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: Ping - old patch from April - mingw support for I32/I64 MS printf formatters to c-format.c


On Wed, 5 Dec 2007, NightStrike wrote:

> On 12/4/07, Joseph S. Myers <joseph@codesourcery.com> wrote:
> > On Tue, 4 Dec 2007, NightStrike wrote:
> >
> > > http://gcc.gnu.org/ml/gcc-patches/2007-04/msg00767.html
> > >
> > > This old thread is in need of review to help the x86_64-pc-mingw32
> > > port of gcc.  It's been about 8 months so far.  Any takers to review
> > > the patch?
> >
> > If there is a patch with full testcases, please point to it - that patch
> > appears to be one without testcases, and so not ready for review.
> 
> I think Kai was waiting until your review before making the testcases.
>  Can you at least evaluate what's there already?

No, in the original discussions leading to that patch we already discussed 
the general nature of what the feature should look like at the language 
level, and assessing whether the patch meets that requires testcases (and 
changes to existing testcases) that provide proper coverage of the new and 
existing features and leave all the format checking tests passing on both 
MinGW and non-MinGW targets.

If after reviewing testcases and code I suspect problems in areas not 
covered by the tests I build the compiler with a patch when reviewing it 
and try my own tests, but the basic tests to provide thorough coverage of 
the patch are the responsibility of the patch submitter.  This (and 
diagnostic-related patches in general) is the sort of patch I review 
primarily by examining the testcases to see if I agree with the 
diagnostics asserted to be present or absent and that the testcases appear 
to cover the feature as described, and only secondarily if the testcases 
appear correct and adequate by examining the implementation.

I made clear on 25 April 
<http://gcc.gnu.org/ml/gcc-patches/2007-04/msg01734.html> that I was 
awaiting testsuite changes.  I do not think there is any open question 
regarding the principle of the feature (where partial patches might be of 
value in assessing how worthwhile a feature is bearing in mind the cost of 
the implementation); what needs review is the quality of an 
implementation, and that needs a complete patch submission.

-- 
Joseph S. Myers
joseph@codesourcery.com


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