On Wed, Oct 03, 2007 at 04:46:00PM -0400, David Turner wrote:
> > > >     if self.need_unicode and not isinstance(query, unicode):
> > > >         try:
> > > >             query = unicode(query, self.encoding)
> > > >         except UnicodeError:
> > > >             pass
> 
> I was thinking that it might make more sense to just remove that whole
> section.  I understand that whoever wrote that code thought it was
> necessary to convert to Unicode, but I think it is not.  Can you comment
> on this idea?

   Yes, that is the idea - there are reports that MySQLdb 1.2.1+ requires
unicode.

Oleg.
-- 
     Oleg Broytmann            http://phd.pp.ru/            [EMAIL PROTECTED]
           Programmers don't die, they just GOSUB without RETURN.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
sqlobject-discuss mailing list
sqlobject-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sqlobject-discuss

Reply via email to