On Friday 20 July 2007 13:44:40 chromatic wrote:

> > Modified: trunk/src/pmc/namespace.pmc
> > =========================================================================
> >== === --- trunk/src/pmc/namespace.pmc       (original)
> > +++ trunk/src/pmc/namespace.pmc     Fri Jul 20 00:47:44 2007
> > @@ -237,7 +237,7 @@
> >                  STRING *meth_name = key;
> >
> >                  if (string_str_index(interp, key,
> > -                    CONST_STRING(interp, "__"), 0 == 0)) {
> > +                    CONST_STRING(interp, "__"), 0) == 0) {
> >                      meth_name = string_substr(interp, key, 2,
> >                          string_length(interp, key) - 2, NULL, 0);
> >                  }

> This checkin breaks t/pmc/freeze.t for me, and I suspect that does bad
> things elsewhere.  I don't have time to investigate right now, but it's
> worth checking!

Fixed in r20065.  I'm not sure *how* the old code (with two bugs) worked, but 
the new code has two fewer bugs and it also works.

-- c

Reply via email to