<div dir="ltr">That's actually a pretty good point,<div>Backwards compatibility in Erlang is usually a slow gradual 3 version process..</div><div>But it might be prudent to throw a warning when a non orddict is introduced. </div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Oct 3, 2013 at 11:27 PM, Rich Neswold <span dir="ltr"><<a href="mailto:rich.neswold@gmail.com" target="_blank">rich.neswold@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="im">On Thu, Oct 3, 2013 at 7:58 PM, Steve Vinoski <span dir="ltr"><<a href="mailto:vinoski@ieee.org" target="_blank">vinoski@ieee.org</a>></span> wrote:<br>

</div><div class="gmail_extra"><div class="im"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">OK, Anthony brought up yet another issue with my previous modified orddict, which is that if someone passes an improper list instead of an orddict, the new version acts differently than the old. Since the goal is backwards compatibility with improved performance, I've added a fourth commit that deals with these issues.</div>


</blockquote></div><div><br></div></div><div>I'm not sure whether this is the backwards compatibility one wants to preserve. Usually you want to preserve an API, but in this case you're preserving code that manipulates the internal representation of an orddict. Any code that uses the orddict API should not end up with an improper list. Any code that passes an improper list to orddict should break.</div>

<span class="HOEnZb"><font color="#888888">
<div><br></div>-- <br>Rich<br>
</font></span></div></div>
<br>_______________________________________________<br>
erlang-patches mailing list<br>
<a href="mailto:erlang-patches@erlang.org">erlang-patches@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/listinfo/erlang-patches</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>/* Sincerely<br>-------------------------------------------------------------- <br>Pedram Nimreezi - Chief Technology Officer  */<div><br></div><div>// The hardest part of design … is keeping features out. - Donald Norman</div>

<div><br></div><div><br></div>
</div>