Nick Glencross <[EMAIL PROTECTED]> wrote:

> On a semi-related note, there's some broken morph code in scalar. It has
> code like:

>     void increment () {
>         PMC_int_val(SELF) = DYNSELF.get_integer() + 1;
>     }

> which is then used by subclasses String, Integer and Float.

Integer and Float have their own implementation. But String inherits
this function, which is wrong, yep.

> In the case of String, the get_integer succeeds,

I'd say that plain String PMCs don't have increment and decrement.

It seems best to just remove scalar.increment and .decrement, which then
would automatically create the default_increment for Strings.

> Nick

leo

Reply via email to