On Fri, Dec 09, 2011 at 03:45:44PM -0500, Wolfgang Denk wrote:
> Dear Jason,
>
> In message <20111209134819.GA26840@jhobbs-laptop> you wrote:
> >
> > > default:
> > > printf("Ignoring malformed menu command: %.*s\n",
> > > (int)(*c - s), s);
> > > + err = -1;
> >
> > err should either be set to 0 here, or initialized to 0 at the top of
> > the function. Setting it to -1 will cause the parser to give up rather
> > than just printing out the warning message. It doesn't have to give up,
> > and not giving up makes the parser more accommodating of pxelinux
> > commands that aren't supported in U-Boot.
>
> You have way more experience with PXE than me, but if we runinto this
> case, doesn't that mean that the whole menu setup is severely broken,
> and continuing is more or less invoking random behaviour?
It only means that an unrecognized menu command was used. It could be
something aesthetic in nature, like a menu title command.
>
> If you really want to see a 0 here, then please feel free to submit an
> updated / fixed patch.
Heiko Schocher ended up sending a patch to do this today, which I've
acked.
Thanks,
Jason
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
