<div dir="ltr">So I submitted a pull request for this now.<br><br>Staffan<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 5, 2014 at 4:02 PM, Mike Gatny <span dir="ltr"><<a href="mailto:mgatny@connamara.com" target="_blank">mgatny@connamara.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="gmail_extra"><div class="gmail_quote"><span class="">On Wed, Nov 5, 2014 at 8:31 AM, Staffan Ulfberg <span dir="ltr"><<a href="mailto:staffan@ulfberg.se" target="_blank">staffan@ulfberg.se</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div><div>Hi,<br><br></div>In quickfixj, fields override the Equals and Hashcode methods which is often useful (I could give examples but I'm sure they are pretty obvious). Not having them is not really a problelm (since Dictionaries etc have constructors that take the IEqualityComparer to use), but for people that migrate from quickfixj (like me) and possible also for people that just assume that Equals on FIX fields "just works" this is confusing.<br><br></div>I would suggest these methods are implemented to perform a comparison of the tag number and value by overriding them in the FieldBase class.<br></div></div></blockquote><div><br></div></span><div>I think this is a good idea.  In general, one of the main goals of qf/n is to allow developers to write idiomatic .NET code.</div><br clear="all"><div><div><div dir="ltr"><div><font face="arial, helvetica, sans-serif">--</font></div><font face="arial, helvetica, sans-serif">Mike Gatny<br></font></div></div></div><div><span style="font-family:arial,helvetica,sans-serif">Connamara Systems, LLC</span></div><div><br></div></div></div></div>
<br>_______________________________________________<br>
Quickfixn mailing list<br>
<a href="mailto:Quickfixn@lists.quickfixn.com">Quickfixn@lists.quickfixn.com</a><br>
<a href="http://lists.quickfixn.com/listinfo.cgi/quickfixn-quickfixn.com" target="_blank">http://lists.quickfixn.com/listinfo.cgi/quickfixn-quickfixn.com</a><br>
<br></blockquote></div><br></div>