<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">On Sat, Oct 23, 2021 at 11:21 PM 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">AFAICS the TMaskOpCode and TMaskOpcodesEnum types are not related at all.<br>
TMaskOpCode is some kind of internal identifier in the internal mask<br>
representation, whereas TMaskOpcodesEnum are options for interpreting<br>
the mask.<br>
<br>
Q1: is it possible to have the TMaskOpCode as an internal type in the<br>
class (maybe in the protected part of the class definition)?<br>
It seems it is not needed outside (as in the public part of the interface).<br>
<br>
If we did not already have the old TMaskOption type, I would actually<br>
have liked to propose to rename the TMaskOpcodesEnum into<br>
TMaskOption...<br></blockquote><div><br></div><div>In 964d5f4d69 I changed most names as you suggested.</div><div>The original TMaskOpCode is now TMaskParsedCode because it is not related to the other enums directly.</div><div>I did the changes before reading your last mail.</div><div>I remember many enums actually are class internals in José's original code (in GitHub). I moved them outside to create the constants more easily. Maybe they can be moved back to a class. You can experiment with it.</div><div>You can rename other stuff, too, if you like. I will not touch it for a while now.</div><div><br></div><div>Juha</div><div><br></div></div></div></div></div></div>