Some comments:<div><br></div><div>garray.c : 204</div><div><div>+<span class="Apple-tab-span" style="white-space:pre">        </span>else {</div><div>+<span class="Apple-tab-span" style="white-space:pre">                </span>// MOONLIGHT_FIXME</div>
<div>+<span class="Apple-tab-span" style="white-space:pre">                </span>// should we handle the case where they want to shrink the array? (note: glib doesn&#39;t)</div><div>+<span class="Apple-tab-span" style="white-space:pre">        </span>}</div>
<div><br></div><div>Either handle it or assert. Actually, I don&#39;t think that anything special must be done to handle shrink,</div><div>not even clear_.</div><div><br></div><div><div>--- a/eglib/src/gbytearray.c</div><div>
+++ b/eglib/src/gbytearray.c</div></div><div><br></div><div>This code is just a cleanup so should be pushed separately, right?</div><div><br></div><div><div>--- a/eglib/src/ghashtable.c</div><div>+++ b/eglib/src/ghashtable.c</div>
</div><div><br></div><div>This change is a fix for compatibility and not new stuff, put it in a separate commit. </div><div><br></div><div><br></div><div><div>+gunichar*</div><div>+g_utf8_to_ucs4_fast (const gchar *str, glong len, glong *items_written)</div>
<div>+{</div></div><div><br></div><div>You g_malloc0 here, it doesn&#39;t make sense to zero since it will be filled right after.</div><div> </div><div class="gmail_quote">On Mon, Apr 18, 2011 at 4:14 PM, Chris Toshok <span dir="ltr">&lt;<a href="mailto:toshok@gmail.com">toshok@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;"><span style="border-collapse:collapse;font-family:arial, sans-serif;font-size:medium"><div style="font-size:13px;margin-top:5px;margin-right:15px;margin-bottom:5px;margin-left:15px;padding-bottom:20px">

<div><span style="border-collapse:collapse;font-family:arial, sans-serif;font-size:13px">Attached is the patch we&#39;re currently working with wrt moonlight running on eglib (needed for our coming embedded dominance, and used in the android port).  Anyone care to review it?  I&#39;d love to commit it today sometime so we can move forward with merging all the rest of the moonlight changes into the mainline.<div>

<br></div><div>There are a couple of FIXME&#39;s in the patch, all of which will be addressed pretty quickly (the exception perhaps being g_unichar_break_type - we&#39;re going to need someone who knows unicode to deal with that mess).</div>

<div><br></div><div>Thoughts, comments?</div><div>Chris</div><div><br></div><font color="#888888"></font></span><font color="#888888"></font></div></div><div style="font-size:13px;margin-top:5px;margin-right:15px;margin-bottom:15px;margin-left:15px;clear:both">

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