Re: Possible new issue introduced with 20140316 release
Mackram Raydan <[email protected]> Wed, 09 Apr 2014 21:18:58 +0300
| Newsgroups | gmane.lisp.clsql.general |
|---|---|
| Message-ID | <[email protected]> |
Hey Russ, Thank you for taking the time to explain your reasoning to me. I actually agree with your argument that the two way operation is a good argument to make the change and as you mention read-from-string is too forgiving to something that in this case should not be. Actually I consider intern to be wiser to the extent that actually to fix the two instances of issues that we got I have actually implemented the quick hack you happened to suggest (ie turn them into string) until I better understand the reasoning behind this. I also happen to believe that your suggestion of the better way of splitting on : or :: (which was something we considered but we needed a quick hack) is the way to go. I am more then happy to do the batch and submit it this weekend with the tests (unfortunately due to my schedule I am rarely able to contribute back to the community except on my weekends). Finally, I would like to thank you and the rest of the team for taking the time to make all these changes and constantly making clsql constantly better for the rest of us. The fact is its quite impressive that so much has changed/refactored and this is the only kind of error we get (and only because I happen to use clsql a bit interestingly :) ). Thank you Mackram