<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">On Sat, Oct 16, 2021 at 12:23 AM Bart via lazarus <<a href="mailto:lazarus@lists.lazarus-ide.org">lazarus@lists.lazarus-ide.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> Yes, there should be an option for backwards compatible syntax, like "[?]" . However the small changes in the syntax can be seen as improvements.<br>
> There already is a define for converting "[z-a]" to "[a-z]".<br>
<br>
Why a define?<br>
Shouldn't that better be configurable at runtime?<br>
</blockquote><div><br></div><div>Yes, it should be a run time option. I will look at it.</div><div>BTW, I forgot the details of "[?]" syntax. I will go through the old mails.<br></div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
>> José's code also catered for some odd DOS behaviour where a mask like<br>
>><br>
>> 'file.txt?' actually matches with the filename 'file.txt' and a mask<br>
>> 'file?.txt' would match 'file1234.txt' (but not 'file12345.txt').<br>
>> That would be rather counter-intuitive to most users.<br>
>> I did not test (it was in one of the "quirks" settings) if it behaves<br>
>> like this by default, but that should IMHO not be the case.<br>
<br>
I tested it now, and by default it returns fals for those cases.<br></blockquote><div><br></div><div>Bart, can you please list cases where the Windows mask behavior differs from actual Windows cmd prompt behavior.</div><div>Adding them to the unit test case would help, too.</div><div>The case with Unit1.pas.bak syntax was fixed already.</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I did not study th enre code at length, but it looks quite a bit more<br>
complicated to me than the old one.<br>
Not sure if I will be able to contribute to it anymore.<br></blockquote><div><br></div><div>It is not so bad. Debugging is difficult because pointer syntax is used, but I remember the old code was difficult to debug as well.</div><div><div>There are 2 essential methods:</div><div> procedure Compile and function Matches().</div></div><div>I refactored the procedure Compile a bit. Originally it was one big block of code.</div><div><br></div><div>Juha</div><div><br></div><div>P.S.</div><div>Not related to the Mask code but I installed TortoiseGit on Windows.</div><div>It works OK but still does not offer anything as good as <b>gitk</b>.</div><div>The "Show log" view shows commits and their messages, and names of changed files but no diffs.</div><div>Getting diffs requires extra mouse clicking for each file separately.</div><div>Then there is "Revision graph" which is actually a branch graph. Not very useful.</div><div>Earlier I have tried SmartGit but I didn't see any benefit using it either.</div><div>"<b>git gui</b>", <b>gitk</b> + some git console commands work best for me.</div><div><br></div></div></div></div></div></div>