<br><br><div class="gmail_quote">On Tue, Jan 18, 2011 at 10:22 AM, Miguel de Icaza <span dir="ltr">&lt;<a href="mailto:miguel@novell.com" target="_blank">miguel@novell.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hello Regan,<br>
<div><br>
&gt; 1) First off the samples do not build on a clean clone of github.<br>
&gt;  Samples.patch fixes that.<br>
<br>
</div>Thanks!<br>
<div><br>
&gt; 2) NSAffineTransform:  Added a missing struct, and finished the bindings for<br>
&gt; this class. In the mac libraries, part of this class is defined in<br>
&gt; Foundation, and part in Appkit (as extension methods).  Since this class is<br>
&gt; only available in OS X, and it all ends up in monomac.dll, I saw no need to<br>
&gt; separate the class. let me know if it should be otherwise.<br>
<br>
</div>Is there a reason why we cant just use the existing CGAffineTransform<br>
class defined in CoreGraphics instead?   It already has a bunch of<br>
useful methods defined, and we could unify some code.<br>
<div></div></blockquote><div><br></div><div>Good catch.  I didn&#39;t use it, because I didn&#39;t see it.  Disregard that patch, and I&#39;ll send you something tomorrow.</div><div><br></div>
<div>  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>
&gt; 3)  NSBezierPath: Added manual bindings to complete the class.  I struggled<br>
&gt; with the packedGlyph array, and decided it was best to expose this as a<br>
&gt; &quot;black box&quot; IntPtr.  It probably won&#39;t be needed from most user code, and if<br>
&gt; you want a packed array (as I do in my sample), you are going to get it as<br>
&gt; an IntPtr from some other cocoa object. I have verified it works correctly.<br>
&gt;  I will provide docs for the manual bindings shortly.<br>
<br>
</div>Is there a reason to drop the CreateBezierPath() method?  Does the<br>
plain constructor just work?<br>
<br></blockquote><div>Yes - I did some testing to confirm, and AFAIK, [NSBezier BezierPath] is a &quot;short cut&quot; for [[NSBezier alloc] init], and provides some symmetry with the other Bezier constructors in cocoa.  In C# it is just redundant (and therefore confusing), which is why I suggest removing it.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I applied the Bezier patch.<br>
<font color="#888888"><br>
Miguel<br>
</font></blockquote></div><br>