<div class="gmail_quote">On Fri, Jun 26, 2009 at 3:20 PM, Boris Smilga <span dir="ltr"><<a href="mailto:boris.smilga@gmail.com">boris.smilga@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;">
Just a minor cavil: does not *STRING-TO-KEY* (or maybe *IDENTIFIER-NAME-TO-KEY*) seem like a better name for this variable?</blockquote><div><br>I would prefer *IDENTIFIER-NAME-TO-KEY* since it gives more context about what the function is used for.  *STRING-TO-KEY* is confusing since it's unclear where the "string" is coming from.<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>
Rationale: I would say that the -JSON- and -LISP- parts of the original *JSON-SYMBOLIZE-LISP-KEY* are not in place here because, unlike *JSON-IDENTIFIER-NAME-TO-LISP*, the function does not translate between JSON and Lisp conventions.  Further, -SYMBOLIZE- is a very imperative form, and imperatives should, in my humble opinion, be limited to functions with side effects.<br>

<br>
What do you think?<br>
<br>
 - B. Sm.<br>
</blockquote><div><br>-Red <br></div></div><br>