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: [RFC] Fixes to bootstrap from a C++ compiler (part 1)


Bernardo Innocenti wrote:

On Friday 25 June 2004 07:09, Richard Kenner wrote:


You are missing numerous blanks.

+ struct md_constant *def = (struct md_constant *)*slot;

Such as after the ")" here and in many similar places.



I've just finished fixing these...


The revised patch is OK, so long as you test it. I agree that these changes actually make the code better, even in C; the C pointer-casting sloppiness was a deliberate concession to existing practice, not a feature.

Further similar patches are pre-approved, so long as you test them as well and so long as your are not making structural changes. We should not expect that other people will necessarily refrain from checking in things that break the copmilation with C++, or blame them for doing that; it's not (yet?) part of our requirements that people worry about that.

Gaby will be disappointed in me, since I opposed his attempt at doing something similar a while back. It's not that I like you better; it's just that I think I made a mistake then.

--
Mark Mitchell
CodeSourcery, LLC
(916) 791-8304
mark@codesourcery.com


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