[Lazarus] Masks: ConstructLegacy

Bart bartjunk64 at gmail.com
Wed Oct 27 13:47:18 CEST 2021


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


More information about the lazarus mailing list