@b4n commented on this pull request.
> @@ -593,8 +593,8 @@ static void print_external(GeanyDocument *doc)
return;
}
- cmdline = g_strdup(printing_prefs.external_print_cmd);
- utils_str_replace_all(&cmdline, "%f", doc->file_name);
+ /* replace d, e, f and p placeholders in cmdline */
+ cmdline = utils_replace_placeholder(doc,
printing_prefs.external_print_cmd, "defp");
See #4319 for migrating the previous values. It'll break for anybody that
already migrated, but that's not gonna be as many users (I guess).
It also fixes the default values, which currently are not aligned with the new
placeholders.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/4250#discussion_r2118303393
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/pull/4250/review/[email protected]>
