Hi,

The new masks unit has several CreateLegacy constructors (and some
*Legacy* functions).
They call the new constructros with mocEscapeChar disabled.

Wouldn't is be a bit more logical to exclude mocEscapeChar form the
MaskOpCodesDefaultAllowed constant, since we'ld like to have the
default behaviour as compatible as possible?

Since most of this code will be used for filename matching, escaping
in most common practice won't be necessary.
Also, in existing code people may have mask with '\' as intentionally
as a PathDelim, and this would break that.

If we leave things as is, then the question remains: how long do we
keep the *Legacy* constructors and functions?

-- 
Bart
-- 
_______________________________________________
lazarus mailing list
lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus

Reply via email to