Re: Possible new issue introduced with 20140316 release

Ralf Mattes <rm-OuoMP45H64u6ogTlOYt/[email protected]> Tue, 8 Apr 2014 15:55:38 +0200
Newsgroups gmane.lisp.clsql.general
Message-ID <[email protected]>
On Tue, Apr 08, 2014 at 04:08:03PM +0300, Mackram Raydan wrote:
> ....
> Can someone please help by shedding some light on why the change was done?

Use your tools  :-)

$ git blame oodml.lisp

 ...
 4f756ab5 sql/oodml.lisp    (Russ Tyndall       2013-06-20 15:12:31 -0400  623)        (when (< 0 (length val))
 4f756ab5 sql/oodml.lisp    (Russ Tyndall       2013-06-20 15:12:31 -0400  624)          (intern (symbol-name-default-case val))))
 4f756ab5 sql/oodml.lisp    (Russ Tyndall       2013-06-20 15:12:31 -0400  625)       ((smallint mediumint bigint integer universal-time)
 ...

So Russ did it in commit 4f756ab5.
$ git log 4f756ab5

 commit 4f756ab532ff033a34597a1c8030379e252952ca
 Author: Russ Tyndall <[email protected]>
 Date:   Thu Jun 20 15:12:31 2013 -0400

    refactored read-sql-value to centralize this logic and reduce
    overloading cases

     * read-eval is off for all read-sql-value cases now
     * the type=symbol case uses intern instead read-from-string

Does this help? 


alf Mattes