This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]

Re: Cosmetic tweaks: cpplib, toplev.c

To: Neil Booth <neil at daikokuya dot demon dot co dot uk>
Subject: Re: Cosmetic tweaks: cpplib, toplev.c
From: "Zack Weinberg" <zackw at Stanford dot EDU>
Date: Tue, 20 Feb 2001 01:12:07 -0800
Cc: gcc-patches at gcc dot gnu dot org
References: <20010219220834.D30485@wolery.stanford.edu> <20010220075350.A12069@daikokuya.demon.co.uk>

On Tue, Feb 20, 2001 at 07:53:50AM +0000, Neil Booth wrote:
> Hi Zack,
> 
> Just a minor nit...
> 
> Zack Weinberg wrote:-
> 
> > 	* cpphash.h (struct cpp_reader): Add print_version field.
> 
> Can we stick to the convention that as much as possible specified
> on the command line goes in the options structure?  I think this is
> 90% true at the moment, and that one or two have gotten away.

Would it ever be useful to set print_version by hand?  That was my
rule of thumb for what went in the options structure.

zw

References:
- Cosmetic tweaks: cpplib, toplev.c
  - From: Zack Weinberg
- Re: Cosmetic tweaks: cpplib, toplev.c
  - From: Neil Booth

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]