[postmodern-devel] Patch: {} syntax for arrays should be inside ''

Stas Boukarev stassats at gmail.com
Tue Feb 15 15:16:59 UTC 2011


Marijn Haverbeke <marijnh at gmail.com> writes:

> Hi Stas,
>
> What happens when, with your patch, you try to pass an array as a
> query parameter? ($1, etc) I suspect the correct thing to do would be
> to return T as a second value from to-sql-string (which indicates that
> the string should be escaped when put into a query), rather than
> putting quotes into the returned string. Also, do you know what the
> correct syntax is when the array contains a string containing a single
> quote? I suspect it should be escaped, which will also be handled by
> returning a second t value. If you could test this for me, I'd be
> thankful.
Indeed, returning T works in all cases you described. I've attached a
corrected patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: arrays-2.diff
Type: text/x-patch
Size: 2898 bytes
Desc: not available
URL: <https://mailman.common-lisp.net/pipermail/postmodern-devel/attachments/20110215/0862b4fb/attachment.bin>
-------------- next part --------------

-- 
With Best Regards, Stas.


More information about the postmodern-devel mailing list