So, note to self, don&#39;t write BCL code when distracted and you should be doing other things. Added a few more tests and fixed Contains and GetEnumerator.<br><br><div class="gmail_quote">On Tue, Nov 10, 2009 at 15:43, ermau <span dir="ltr">&lt;<a href="mailto:me@ermau.com">me@ermau.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">In reviewing removing the else, I realized there was a bug to begin with. Made your other requested changes, added a test for the bug and fixed it.<div>
<div></div><div class="h5"><div><br><div class="gmail_quote">On Tue, Nov 10, 2009 at 13:36, Jb Evain <span dir="ltr">&lt;<a href="mailto:jb@nurv.fr" target="_blank">jb@nurv.fr</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey,<br>
<div><div></div><div><br>
On 11/10/09, ermau &lt;<a href="mailto:me@ermau.com" target="_blank">me@ermau.com</a>&gt; wrote:<br>
&gt; .NET supports null keys for groupings in<br>
&gt; Enumerable.ToLookup()/Lookup&lt;&gt;, here&#39;s a patch for review<br>
&gt; to improve Mono compatibility.<br>
<br>
</div></div>+                       Assert.IsTrue (l[null].Contains (&quot;2&quot;));<br>
<br>
Please add a space before indexing, like you do before calling a method.<br>
<br>
+                               }<br>
+                               else if (!dictionary.TryGetValue (key, out list)) {<br>
<br>
Put the else on the same line as the }<br>
<br>
+                               if (key == null &amp;&amp; nullGrouping != null)<br>
+                                       return nullGrouping;<br>
+                               else<br>
+                               {<br>
+                                       IGrouping&lt;TKey, TElement&gt; group;<br>
+                                       if (groups.TryGetValue (key, out group))<br>
+                                               return group;<br>
+                               }<br>
<br>
Remove the else and move the code to the same level as the if.<br>
<br>
When it&#39;s done please go ahead and commit to trunk and mono-2-6.<br>
<br>
Thanks!<br>
<font color="#888888"><br>
--<br>
Jb Evain  &lt;<a href="mailto:jb@nurv.fr" target="_blank">jb@nurv.fr</a>&gt;<br>
</font></blockquote></div><br></div>
</div></div></blockquote></div><br>