Re: [Lazarus] MatchesWindowsMask('[x]','[x]') return False

2020-03-15 Thread Mattias Gaertner via lazarus
On Mon, 16 Mar 2020 00:21:55 +0100 Marco van de Voort via lazarus wrote: > Op 2020-03-16 om 00:05 schreef Mattias Gaertner via lazarus: > >> Using \ as an escape character isn't really a good choice IMO, > >> since this function deals with filenames. > > So? It is common practice on Unix since

Re: [Lazarus] MatchesWindowsMask('[x]','[x]') return False

2020-03-15 Thread Marco van de Voort via lazarus
Op 2020-03-16 om 00:05 schreef Mattias Gaertner via lazarus: Using \ as an escape character isn't really a good choice IMO, since this function deals with filenames. So? It is common practice on Unix since decades. Well, euhh, Unix doesn't use backslashes in paths? This would require

Re: [Lazarus] MatchesWindowsMask('[x]','[x]') return False

2020-03-15 Thread Mattias Gaertner via lazarus
On Sun, 15 Mar 2020 22:19:21 +0100 Bart via lazarus wrote: > On Sun, Mar 15, 2020 at 8:23 PM Mattias Gaertner via lazarus > wrote: > > > IMO it should check for \[ > > Using \ as an escape character isn't really a good choice IMO, since > this function deals with filenames. So? It is

Re: [Lazarus] MatchesWindowsMask('[x]','[x]') return False

2020-03-15 Thread Bart via lazarus
On Sun, Mar 15, 2020 at 8:23 PM Mattias Gaertner via lazarus wrote: > IMO it should check for \[ Using \ as an escape character isn't really a good choice IMO, since this function deals with filenames. I would be more inclined to have an option to disable sets in the mask. @Rolf: you can do a

Re: [Lazarus] MatchesWindowsMask('[x]','[x]') return False

2020-03-15 Thread Mattias Gaertner via lazarus
On Sun, 15 Mar 2020 13:09:55 +0100 Rolf Wetjen via lazarus wrote: > Hi Lazarus team, > > I found this during some error analysis. TMask uses the brackets as a > set indicatior in the mask string. In this case the FMask.MinLength > and FMask.MaxLength are set to 1. The brackets in the test

Re: [Lazarus] MatchesWindowsMask('[x]','[x]') return False

2020-03-15 Thread Rolf Wetjen via lazarus
Hi Bart, sorry, this isn't a workaround if you get the filenames form FindFirst/FindNext and want to test against a fixed mask. My workaround for MatchesWindowsMask is quick' n'dirty too: Replace [ with #17 and ] with #18 in the mask and in the test string. Anyway, let's wait for the info

Re: [Lazarus] MatchesWindowsMask('[x]','[x]') return False

2020-03-15 Thread Bart via lazarus
On Sun, Mar 15, 2020 at 2:36 PM Bart wrote: > Somebody should test this behaviour against Delphi. I asked on the Dutch Delphi forum. -- Bart -- ___ lazarus mailing list lazarus@lists.lazarus-ide.org https://lists.lazarus-ide.org/listinfo/lazarus

Re: [Lazarus] MatchesWindowsMask('[x]','[x]') return False

2020-03-15 Thread Bart via lazarus
On Sun, Mar 15, 2020 at 1:10 PM Rolf Wetjen via lazarus wrote: > I found this during some error analysis. '[[]x]' as a mask will match '[x]' as filename. So at least there is a workaround. -- Bart -- ___ lazarus mailing list

Re: [Lazarus] MatchesWindowsMask('[x]','[x]') return False

2020-03-15 Thread Bart via lazarus
On Sun, Mar 15, 2020 at 2:36 PM Bart wrote: > Somebody should test this behaviour against Delphi. PS. I can't since D7 (the only Delphi I have) doesn't have MatchesMask. -- Bart -- ___ lazarus mailing list lazarus@lists.lazarus-ide.org

[Lazarus] MatchesWindowsMask('[x]','[x]') return False

2020-03-15 Thread Rolf Wetjen via lazarus
Hi Lazarus team, I found this during some error analysis. TMask uses the brackets as a set indicatior in the mask string. In this case the FMask.MinLength and FMask.MaxLength are set to 1. The brackets in the test string are not treated in any special way and so the match fails as the length