@b4n commented on this pull request.
> + * @param[in] needles The string with placeholders to replace. + * The string is a combination of 'd', 'e', 'f', 'l' and 'p'. + * 'd' : replace %d with the absolute path + * 'e' : replace %e with the filename (excluding extension) + * 'f' : replace %f with the filename (including extension) + * 'l' : replace %l with the current 1-based linenumber + * 'p' : within a project context: + * replace %p with the absolute project base directory + * within a document context: + * replace %p with the absolute path/filename (including extension) + * + * e.g. 'df' will replace all %d and %f placeholders What is the goal of this parameter? AFAIK, it is only used to exclude `%l` from the external print command, which doesn't seem very much required to me (I don't see how it would be a problem -- and it could even be seen as useful, e.g. to only print around the caret or some other hypothetical use case). -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/4250#pullrequestreview-2884579034 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany/pull/4250/review/2884579...@github.com>
