And it turns out this works great!  We now pass all of the tests in
test_str and test_unicode related to calling str/unicode and getting
the appropriate __str__/__unicode__ method called.  We also now properly
don't decode strings when you do unicode('foo') so we no longer throw
for Vernon's exception case where CPython doesn't throw.

So I'm inclined to go with this fix.

> -----Original Message-----
> From: [email protected] [mailto:users-
> [email protected]] On Behalf Of Dino Viehland
> Sent: Wednesday, February 10, 2010 9:38 AM
> To: Discussion of IronPython
> Subject: Re: [IronPython] Django, __unicode__, and #20366
> 
> It definitely won't be in 2.6RC1 but could be in 2.6 final (or RC2 or
> whatever).
> 
> I'm still looking into Michael's proposed fix of recognizing calls to
> unicode -
> it's a little tricky dealing with * and **args calls to it and I got
> distracted
> by preparing for PyCon.
> 
> Hopefully I can finish that up in the next couple of days and see how
> that
> works vs. just calling __unicode__ when __str__ gets called.
> 
> > -----Original Message-----
> > From: [email protected] [mailto:users-
> > [email protected]] On Behalf Of Jeff Hardy
> > Sent: Wednesday, February 10, 2010 9:21 AM
> > To: Discussion of IronPython
> > Subject: Re: [IronPython] Django, __unicode__, and #20366
> >
> > On Mon, Feb 1, 2010 at 2:46 PM, Dino Viehland <[email protected]>
> wrote:
> > > Messing with identity starts to get really scary and I'd rather not
> go
> > > there - I'm sure there will be lots of edge cases which will be
> broken.
> > >
> > > I could see is making unicode(foo) do something different.  If you
> aliased
> > > unicode then you'd get str's behavior though but that might be
> perfectly
> > > acceptable.  It's definitely a solution I had not considered and
> it'd
> > > probably fix multiple issues.
> >
> > Hi Dino,
> > It looks like this is a major impediment to running Django; the
> > majority of the test suite failures are because IronPython calls
> > __str__ instead of __unicode__. Any chance this could be fixed for
> > 2.6.1? I know it's a big change, but IMHO it's pretty critical as
> > well.
> >
> > - Jeff
> > _______________________________________________
> > Users mailing list
> > [email protected]
> > http://lists.ironpython.com/listinfo.cgi/users-ironpython.com
> _______________________________________________
> Users mailing list
> [email protected]
> http://lists.ironpython.com/listinfo.cgi/users-ironpython.com

_______________________________________________
Users mailing list
[email protected]
http://lists.ironpython.com/listinfo.cgi/users-ironpython.com

Reply via email to