I think it's significant, though admittedly minor, because readable<div>JS is one of the goals of the project.</div><div><br></div><div>Speaking of readable JS, by far the biggest problem I encounter</div><div>is when PS bundles a complex expression into comma-delimited</div>

<div>Javascript like "(foo(),bar(),baz())" which is basically a progn</div><div>that evaluates to baz(). These things often create monstrous</div><div>one-liners. I don't have a good idea of what to do about it --</div>

<div>I recall that Vladimir tried to pretty-print them at one point</div><div>and this turned out harder than it seemed -- but it's definitely</div><div>the #1 thing that would prevent me from trying to persuade</div>

<div>a JS programmer that my compiled PS is readable.</div><div><br><div><br><div class="gmail_quote">On Thu, Jan 17, 2013 at 9:44 AM, Boris Smilga <span dir="ltr"><<a href="mailto:boris.smilga@gmail.com" target="_blank">boris.smilga@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Also, there is this baddie:<div class="im"><br>
<br>
On 16 Jan 2013, at 00:18, David Sargeant wrote:<br>
<br>
</div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
(function () {<br>
    var collect66 = [];<br>
    for (var num = 10; num <= 1; num += 1) {<br>
        collect66['push'](num);<br>
</blockquote></div>
^^^^^^^^^^^^^^^^^^^^^^^^^^<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    };<br>
    return collect66;<br>
})();<br>
</blockquote>
<br>
Not at all a major issue, but collect66.push would fit rather better in this context.  There are a couple more instances of this kind in the code of ParenScript, they all boil down to keywords being used with @ instead of literal symbols:<br>


<br>
src/lib/ps-dom.lisp:6:  `(@ ,el :inner-h-t-m-l))<br>
src/lib/ps-dom.lisp:12:  `((@ ,el :get-attribute) ,attr))<br>
src/lib/ps-loop.lisp:102:         (then (if (numberp by) `((@ ,var :slice) ,by) `(,by ,var))))<br>
src/lib/ps-loop.lisp:167:        (:collect `((@ ,var :push) ,item))<br>
src/macros.lisp:177:                     (if (eql ,funobj (@ __PS_MV_REG :tag))<br>
src/macros.lisp:178:                         (@ __PS_MV_REG :values)<br>
src/non-cl.lisp:189:      `((@ (list ,@things) :join) "")))<br>
<br>
Do you think we should fix them, or is this totally insignificant?<br>
<br>
 — B. Smilga.<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
______________________________<u></u>_________________<br>
parenscript-devel mailing list<br>
<a href="mailto:parenscript-devel@common-lisp.net" target="_blank">parenscript-devel@common-lisp.<u></u>net</a><br>
<a href="http://lists.common-lisp.net/cgi-bin/mailman/listinfo/parenscript-devel" target="_blank">http://lists.common-lisp.net/<u></u>cgi-bin/mailman/listinfo/<u></u>parenscript-devel</a><br>
</div></div></blockquote></div><br></div></div>