<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><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 class="im">On Wed, Jun 13, 2012 at 1:25 PM, Ian Daniher <<a href="mailto:ian@nonolithlabs.com">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>