Me too. Standardize on all lower-case. Case-insensitive compares/lookups always require conversion and are therefore ALWAYS SLOWER, even if it may only be marginally.<br><br><div><br><div class="gmail_quote">On Wed, Jun 13, 2012 at 3:10 PM, Ian Daniher <span dir="ltr"><<a href="mailto:ian@nonolithlabs.com" target="_blank">ian@nonolithlabs.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="text-align:left">Making the parser case-insensitive breaks the precedent established by the Python and Javascript JSON parsers. That said, I can't think of a reason why you'd have "Formula" and "formula" in a cjson object and case-insensitive is more durable.</div>

<div style="text-align:left"><br></div><div style="text-align:left">My vote is for lowercase standardization.</div><div class="HOEnZb"><div class="h5"><br><div class="gmail_quote">On Wed, Jun 13, 2012 at 1:54 PM, Kyle Lutz <span dir="ltr"><<a href="mailto:kyle.lutz@kitware.com" target="_blank">kyle.lutz@kitware.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Wed, Jun 13, 2012 at 1:25 PM, Ian Daniher <<a href="mailto:ian@nonolithlabs.com" target="_blank">ian@nonolithlabs.com</a>> wrote:<br>


> If you're going to have "inchi" instead of "InChI", you might as well go all<br>
> the way. It'd be great to see line 68 of chemjsonfileformat.cpp fixed.<br>
<br>
</div>This should be an easy fix. I just need to know what the right fix is<br>
:-). I think we should to select one of the following options:<br>
<br>
1. Standardize on all lower-case keys (e.g. always "inchi" instead of<br>
"InChI" and "chemical json" instead of "Chemical JSON").<br>
2. Specify that all keys are to be parsed in a case-insensitive manner<br>
(e.g. "inchi", "InChI", and "INCHI" would all be valid).<br>
3. Some other way I haven't though of...<br>
<br>
Thoughts?<br>
<br>
Cheers,<br>
Kyle<br>
</blockquote></div><br>
</div></div><br>_______________________________________________<br>
Openchemistry-developers mailing list<br>
<a href="mailto:Openchemistry-developers@openchemistry.org">Openchemistry-developers@openchemistry.org</a><br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/openchemistry-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/openchemistry-developers</a><br>
<br></blockquote></div><br></div>