<div class="gmail_quote">On Wed, May 6, 2009 at 1:34 PM, Daniel Gackle <span dir="ltr"><<a href="mailto:danielgackle@gmail.com">danielgackle@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
One thing I forgot to point out: notice that, in the latest PS, the minus operator '- is actually converted into an empty string in JS, which is obviously wrong. <br></blockquote><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="h5"><br><br><div class="gmail_quote">On Wed, May 6, 2009 at 2:27 PM, Daniel Gackle <span dir="ltr"><<a href="mailto:danielgackle@gmail.com" target="_blank">danielgackle@gmail.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">The latest PS interprets QUOTE of a symbol to mean, "first convert this symbol to a Javascript identifier, then a string". This breaks our code, because our formula parser relies on symbols to represent operators. Here's a line of code that PS was generating for us nicely before:<br>





<br>(defparameter *op-symbols* '(| | + - * / ^ < > = <> <= >= & % ! { } [ ] |(| |)| |:| |,| |"| |'|))<br>=><br>var OPSYMBOLS = [' ', '+', '-', '*', '/', '^', '<', '>', '=', '<>', '<=', '>=', '&', '%', '!', '{', '}', '[', ']', '(', ')', ':', ',', '"', '\''];<br>





<br>In the new PS, it looks like this:<br><br>var OPSYMBOLS = [' ', 'plus', '', 'star', 'slash', '^', '<', '>', 'equals', '<>', '<equals', '>equals', '&', 'percent', 'bang', '{', '}', '[', ']', '(', ')', 'colon', ',', '"', '\''];<br>




<br>I think this is a mistake. The only reason for translating symbols like + to "plus" is if you want to use them in a JS identifier. But quoted symbols are not meant to turn into JS identifiers, only JS strings. <br>
</blockquote></div></div></div></blockquote><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div class="h5"><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>Looking at the git log, this change was introduced into special-forms.lisp by commit dd4442b8973fe8b2c19b44f94f244934aa418ae8. This was submitted at the time as a bug fix. From our point of view, this "fix" introduces a bug, and quite a serious one, since it means we can't run our formula parser (and a few other things) in the browser.</blockquote>
</div></div></div></blockquote><div><br>The problem was that (slot-value object
'quoted-slot) had inconsistent semantics compared to other uses of
quoted symbols.  This was fixed by making a quoted symbols behave as
you describe: first convert this symbol to a Javascript identifier,
then a string.<br><br>For an example of where string-translation can go wrong, see this code:<br><br>(let* ((our-slot 'foo-bar)<br>         (val1 (slot-value object our-slot))<br>         (val2 (slot-value object 'foo-bar))<br>
   ...)<br><br>The fix made this code execute as you would believe it should: so that val1 and val2 are equal.  If we were to translate symbols to their STRING-VALUE, val1 would = object["foo-bar"] and val2 would = object.fooBar.  That is, unless you change the semantics of slot-value.<br>
<br>I am not sure what the context of your *op-symbols* code is, but it seems like you should be able to avoid this problem by using strings.  I don't know what the alternative is for the foo-bar example above, if we adopt a symbol translator like you suggest.<br>
<br>Best,<br>Red<br><br></div><div><br><br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>_______________________________________________<br>

parenscript-devel mailing list<br>
<a href="mailto:parenscript-devel@common-lisp.net">parenscript-devel@common-lisp.net</a><br>
<a href="http://common-lisp.net/cgi-bin/mailman/listinfo/parenscript-devel" target="_blank">http://common-lisp.net/cgi-bin/mailman/listinfo/parenscript-devel</a><br>
<br></blockquote></div><br>