<div>I guess the question is 'What do consumers of the JSON parser expect when parsing names?' I really don't know enough about JSON to argue one way or the other on which is correct, but if consumers expect Symbols, then returning Strings will break some code elsewhere.</div>
<div> </div>
<div>I will need to retract the fix for Issue 240 to match the new behavior. But I guess a new fix for 240 is needed to ensure the JSON parser is really returning Strings everywhere.</div>
<div><br clear="all">John O'Keefe [|], Principal Smalltalk Architect, Instantiations Inc.<br><br><br></div>
<div class="gmail_quote">On Fri, Nov 14, 2008 at 1:37 AM, Lukas Renggli <span dir="ltr"><<a href="mailto:renggli@gmail.com">renggli@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">On Fri, Nov 14, 2008 at 7:00 AM, Philippe Marschall<br>
<div class="Ih2E3d"><<a href="mailto:philippe.marschall@gmail.com">philippe.marschall@gmail.com</a>> wrote:<br></div>
<div class="Ih2E3d">> 2008/11/14 Lukas Renggli <<a href="mailto:renggli@gmail.com">renggli@gmail.com</a>>:<br>>> Thank you for fixing the test. I was not paying attention to that when<br>>> writing this.<br>
>><br>>> Wouldn't it be simpler if the JSON parser would parse everything as a<br>>> string and not convert certain values to a symbol?<br>><br>> +1<br><br></div>Name: Javascript-Core-lr.13<br>
Author: lr<br>Time: 14 November 2008, 7:36:49 am<br>UUID: fba167ec-7162-421a-85f4-a8019b30ae89<br>Ancestors: Javascript-Core-lr.12<br><br>- don't parse JSON names to symbols (please check if this passes<br>agains Javascript-Tests-Core-jf.15.mcz)<br>
<div class="Ih2E3d"><br>--<br>Lukas Renggli<br><a href="http://www.lukas-renggli.ch/" target="_blank">http://www.lukas-renggli.ch</a><br>_______________________________________________<br></div>
<div>
<div></div>
<div class="Wj3C7c">seaside-dev mailing list<br><a href="mailto:seaside-dev@lists.squeakfoundation.org">seaside-dev@lists.squeakfoundation.org</a><br><a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" target="_blank">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br>
</div></div></blockquote></div><br>