On 12 April 2007 15:59, Thomas Neumann wrote:

> Hi,
> 
> the attached patch adds missing casts in gengtype-lex.l (i.e. missing in
> C++, recommended by GCC Coding Conventions in C). Please apply it if you
> find it useful.
> 
> Thomas
> 
> * gengtype-lex.l: Cast from void* to the proper type.

  It's good of you to pay attention to this, as we do have a long-range goal
of making gcc compilable in C++, but even though they're close to obviously
correct, it's possible that nobody would feel confident to apply them
'untested'.

  Maybe it would make more sense to bundle them up into two tranches, one for
all the gen* utilities, one for the compiler core itself.  That would be much
more practical to do the full bootstrap-and-regtest procedure.  However,
bundling them all up into big patches would probably run over the size limit
for "small patches that don't require paperwork".  Do you have an assignment
on file with the FSF?

  Meanwhile, since nobody's commented on the -patches list, I'm posting this
across to the main list to request advice.


    cheers,
      DaveK
-- 
Can't think of a witty .sigline today....

Reply via email to