Gregor N. Purdy <[EMAIL PROTECTED]> wrote:
> At line 679 of languages/imcc/pbc.c, we have:


>         case 'I':
>             if (r->name[0] == '0' && r->name[1] == 'x')
>                 r->color = strtoul(r->name+2, 0, 16);
>             else if (r->name[0] == '0' && r->name[1] == 'b')
>                 r->color = strtoul(r->name+2, 0, 2);
>             else
>                 r->color = atoi(r->name);
>             break;

> Shouldn't atoi() be atol()?

Yep.  struct _SymReg defines C<color> as C<int>. This should probably be
an INTVAL, able to store all possible int constants.

> Regards,

> -- Gregor

leo

Reply via email to