Hello,<div><br></div><div>    Thanks fro the great patch. </div><div><br></div><div>    Some small feedback:</div><div><br></div><div>* Boolean values many times are just parsed as &quot;flag&quot;, but this does not give enough context in the parameter, so in these cases it is best to look at the selector name, and replace &quot;flag&quot; with the originals seletor name like &quot;willMove&quot;</div>
<div><br></div><div>* Factory methods (static methods that create instances of an object) most of the time will be called FromXXXX, for example MyWindow FromData (NSData)</div><div><br></div><div>Miguel.<br><br><div class="gmail_quote">
On Fri, Apr 23, 2010 at 1:38 PM, James Clancey <span dir="ltr">&lt;<a href="mailto:james.clancey@gmail.com">james.clancey@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div>Here goes another update. </div><div>#import &lt;AppKit/NSAnimationContext.h&gt;<span style="white-space:pre">                </span>Done</div><div>#import &lt;AppKit/NSFontPanel.h&gt;<span style="white-space:pre">                        </span>Done</div>

<div>#import &lt;AppKit/NSCachedImageRep.h&gt;<span style="white-space:pre">                </span>Done</div><div>#import &lt;AppKit/NSCIImageRep.h&gt;<span style="white-space:pre">                        </span>Done</div>
<div>#import &lt;AppKit/NSCustomImageRep.h&gt;<span style="white-space:pre">                </span>Done</div><div>#import &lt;AppKit/NSEPSImageRep.h&gt;<span style="white-space:pre">                </span><span style="white-space:pre">        </span>Done</div>

<div><br></div><div><div><br></div></div><div><span style="white-space:pre"></span></div>
<br>_______________________________________________<br>
Mono-osx mailing list<br>
<a href="mailto:Mono-osx@lists.ximian.com">Mono-osx@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-osx" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-osx</a><br>
<br></blockquote></div><br></div>