bison-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: push parser


From: Paul Eggert
Subject: Re: push parser
Date: Tue, 12 Dec 2006 15:07:03 -0800
User-agent: Gnus/5.1008 (Gnus v5.10.8) Emacs/21.4 (gnu/linux)

"Joel E. Denny" <address@hidden> writes:

> I would appreciate Paul Eggert's opinion on arguments with multiple 
> const's like this one:
>
>   const YYSTYPE * const yynlval
>
> I'm pretty sure he'll at least want to see the first const moved:
>
>   YYSTYPE const * const ynlval
>
> He may want to see the second const removed.

Yes to both cases, if it's a local variable or parameter.  I prefer
putting 'const' after the type it modifies, for consistency; and for
local vars I generally prefer omitting a final 'const' as it clutters
the code for too-small an advantage.

Sorry I haven't had time to read all the patches recently....




reply via email to

[Prev in Thread] Current Thread [Next in Thread]