Pynckels left a comment (geany/geany#4250) Up until now:
**_New `utils_replace_placeholder`_** Reason: to get a functional logic that can be used from build.c as well as printing.c some changes were necessary. For instance: build can use the placeholder %l, on the other hand this placeholder does not make sense in printing.c **_build.c is changed_** Reason: uses `utils_replace_placeholder` in stead of `build_replace_placeholder`. **_printing.c is changed_** Reason: uses `utils_replace_placeholder` to extend the placeholder replacement with new options. The called utils function allocates the returned string. **_utils.c is changed_** Reason: common functionality between build.c and printing.c is refactored to `utils_replace_placeholder`. The function is documented. It still uses the `app` global variable which is now documented above the function. **_utils.h is changed_** Reason: added declaration of `utils_replace_placeholder` I think this resolves your remarks above, but don't hesitate to let me know if there are other elements. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/4250#issuecomment-2779706593 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany/pull/4250/c2779706...@github.com>
