On Tue, 19 Oct 2021 at 05:58, Richard Kimberly Heck <rikih...@lyx.org>
wrote:

> On 10/18/21 20:37, Thibaut Cuvelier wrote:
>
> Thank you Pavel and Riki for your feedback!
>
> Here is a refactoring of this code. As a result, the execution should be a
> tad slower: sanitizeFileName performs a copy of the string (instead of
> mutating its argument), while the original version did not, but it's only
> for file names, so it should not matter much (especially as the calling
> code would not be too strange).
>
> Is the code OK to push as is?
>
> Looks good to me.
>
Thanks, I've pushed it.
-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel

Reply via email to