<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">On Sun, Oct 17, 2021 at 1:10 AM Bart via lazarus <<a href="mailto:lazarus@lists.lazarus-ide.org" target="_blank">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">I see you implemented the "old" constructors with the TMaskOptions<br>
parameter and deprecated them as suggested.<br>
Thanks for that.<br>
Maybe add: 'Will be removed in 2.4' or similar to the deprecated message?<br>
The corresponding Create in TMaskBase is not deprecated: is that by design?<br></blockquote><div><br></div><div>Actually this will be published in 2.4 initially. After that the deprecated stuff can be removed.</div><div>I added comments.</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">
Thanks also for implementing MaskOpCodesDisableRange<br>
It needs to be apllied in the TShell* components, to presevere default<br>
behaviour.<br></blockquote><div><br></div><div>ShellCtrls.pas was updated already in the initial commit d7036bb0009 last wednesday.</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">
The method TMask.MatchesWindowsMask still is not implemented, so it<br>
immediately causes compiler errors in my code.<br>
This also needs to be implemented an deprecated IMHO.<br></blockquote><div><br></div><div>Ok, yes. I implemented it in c28cc988ec. It requires a temporary TMaskUTF8Windows instance because the syntax is decided in creation.</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">
What does eMaskOpcodeOptionalChar do?<br>
It's not very clear to me from the comments in the code.<br></blockquote><div><br></div><div>Good question. It is the one syntax I don't fully understand yet.</div><div>Below is an explanation from José himself. I will study this syntax later and add more unit test cases.</div><div>---</div><div>eMaskOpcodeOptionalChar is the set "1[ab]2" matches "1a2" and "ab2" but not "1c2".<br></div><div>---</div><div>I am not sure if it is correct. I will return to this later...</div><div><br></div><div>Juha</div><div><br></div></div></div></div></div></div></div>