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]
Other format: [Raw text]

Re: Patch for C++ bug 24009


Mike Stump <mrs@apple.com> writes:

| Looks good.  Read on for some exceedingly minor issues...
| 
| On Aug 26, 2006, at 5:46 AM, Joseph S. Myers wrote:
| > +  gcc_assert (!input_file_stack_restored);
| > +  if (input_file_stack_tick == 0x7fffffff)
| > +    sorry ("GCC supports only %d input file changes",
| > input_file_stack_tick);
| 
| How about clamping at the max and using an empty stack at the max,
| that way, we can still compile programs (don't have to document or
| explain the arbitrary limit), while only sacrificing diagnostic
| fidelity past a whole lot of changes.  I don't think this is a big
| deal as the limit is kinda high.

>From QoI viewpoint, if we put an arbitrary limit of that sort, we
should document it.

-- Gaby


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