[U-Boot-Users] [DTC PATCH 1/2] Add yyerrorf() for formatted error messages.

David Gibson
2007-12-22T02:43:53Z

On Fri, Dec 21, 2007 at 03:28:56PM -0600, Scott Wood wrote:
> On Fri, Dec 21, 2007 at 11:04:39AM +1100, David Gibson wrote:
> > On Thu, Dec 20, 2007 at 10:48:23AM -0600, Scott Wood wrote:
> > > Signed-off-by: Scott Wood <scottwood@freescale.com>
> >
> > No need for a new function. If yyerror() is defined as a varargs
> > function it's still compatible with bison's built-in usage.
>
> Not if yyerror() is called with a percent symbol in the string, which
> looks possible if a percent token is added and verbose syntax errors are
> enabled.

Ah, yes, I guess so.

However from discussion on the other thread, it looks like this
extended yyerror() isn't really what you want. Maybe instead we
should look a more general error/warning printing function, and make
yyerror() call that.

--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!