<div dir="ltr"><div dir="auto">Very nice! Some comments and questions:</div><div><br></div><div>PreferenceNG class>>#detectType: is intended to be used for importing the old preferences and will later be removed? I see it is inconsistent with the initialization method that just sets type to 'aValue class' if aType is nil. To ensure consistency, the initialization method could call #detectType:, unless it's going to be removed. This is specially important for Booleans, otherwise the preference will be stuck in true or false. And should we set the type to the value class or species? Anyway, I'm not sure it's a good idea to do this automatic detection of type, maybe it would be better to just leave it in nil.</div><div><br></div><div>Also, PreferenceNG>>#printOn: capitalizes the name of the preference. Since this name is used as a key to identify the preference, I think it's better to show it as it is without capitalization.</div><div><br></div></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 5 May 2022 at 3:31 AM Hilaire Fernandes via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st" target="_blank">cuis-dev@lists.cuis.st</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF">
    <p><font size="4" style="color:rgb(0,0,0)">Hi, <br>
      </font></p>
    <p><font size="4" style="color:rgb(0,0,0)">This is the third iteration of the class
        PreferencesNG.</font></p>
    <p><font size="4" style="color:rgb(0,0,0)">Most of the preferences/parameters found in
        Preferences/Preferences are implemented. There are a few
        exceptions like the "file list" operation, some behaviors could
        be implemented in the FileList.</font></p>
    <p><font size="4" style="color:rgb(0,0,0)">Regarding metrics: Preferences/Preference are 180
        methods, 942 lines of code; PreferencesNG 42 methods, 402 lines
        of code.<br>
      </font></p></div><div bgcolor="#FFFFFF">
    <p><font size="4" style="color:rgb(0,0,0)"></font>Hilaire<br>
    </p>
    <pre cols="72" style="font-family:monospace">-- 
GNU Dr. Geo
<a href="http://drgeo.eu" style="font-family:monospace" target="_blank">http://drgeo.eu</a>
<a href="http://blog.drgeo.eu" style="font-family:monospace" target="_blank">http://blog.drgeo.eu</a></pre>
  </div>

-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div></div>