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: Bootstrap broken for more than 10 days on cygwin and mingw


FX <fxcoudert@gmail.com> writes:

> Ian, I know you've committed quite a bit of work to introduce C++
> compat warnings, could you please take time to review this simple
> patch?

Sorry for the delay.  You can usually reach me faster at
iant@google.com, per the ChangeLog entry.  I had already looked at the
patch and made the same comment that Michael did: to use the enum
constants rather than the casts.  I probably could have said that more
clearly.  My apologies.

> Also, it would seem reasonable to me if, for very intrusive patches
> such as enabling new warnings to bootstrap, they were tested on
> primary and secondary platforms.

It's not really feasible for me to bootstrap every primary and
secondary platform.  In any case I do not think such a requirement is
appropriate for stage 1.  I am certainly committed to fixing any
problem as soon as it is brought to my attention; in this case there
was already a patch, and I already reviewed it, so I didn't feel that
it was necessary for me to do anything further.

Ian


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