On Mon, Apr 18, 2011 at 12:28 PM, Rodrigo Kumpera <span dir="ltr">&lt;<a href="mailto:kumpera@gmail.com">kumpera@gmail.com</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Some comments:<div><br></div><div>garray.c : 204</div><div><div>+<span style="white-space:pre-wrap">        </span>else {</div><div>+<span style="white-space:pre-wrap">                </span>// MOONLIGHT_FIXME</div>
<div>+<span style="white-space:pre-wrap">                </span>// should we handle the case where they want to shrink the array? (note: glib doesn&#39;t)</div><div>+<span style="white-space:pre-wrap">        </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></blockquote><div><br></div><div>nod.  not much has to happen - i just left the fixme in because there was stuff we might want to address.  I can remove the fixme and leave the other comment, or remove the else branch entirely.</div>
<div><br></div><div>glib does clear the memory in gc-friendly mode, but that&#39;s all they do (and that&#39;s the only time they do it.)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<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></blockquote><div><br></div><div>K, I can split these out.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></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></blockquote><div><br></div><div>Cool, thanks.  Fixed locally.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div class="gmail_quote"><div class="im">On Mon, Apr 18, 2011 at 4:14 PM, Chris Toshok <span dir="ltr">&lt;<a href="mailto:toshok@gmail.com" target="_blank">toshok@gmail.com</a>&gt;</span> wrote:<br>

</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div></div><div class="h5"><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></div></div><div class="im">_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">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></div></blockquote></div><br></div>
</blockquote></div><br>