On Tue, 14 May 2002 01:02:52 -0700 (PDT)
"Sean 'Shaleh' Perry" <[EMAIL PROTECTED]> wrote:
> This is my fault. I broke the code involved when I moved blackbox
> from char[] to std::string. Just committed a fix in cvs.
>
> My problem was the old code did this:
>
> case nop:
> menu->insert(label);
>
> but label may have been all zeros unless one was defined. So [nop] ()
> would crash blackbox because std::string expects a null terminated
> string. To fix this I changed it to:
>
> menu->insert("");
>
> which made the assumption all no ops were merely spacers. So this
> time i fixed the real problem which was invalid strings:
>
> if (! *label)
> label[0] = '\0';
> menu->insert(label);
>
> Thanks for mentioning this.
>
> P.S. an alpha4 should be out by weeks end. Lots of new pretty code.
Yeah, the CVS works great, thanks. Can't wait til' alpha4!
