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: [c/c++] PR 36513: add testcase


On Fri, 2010-02-19 at 00:38 +0100, Manuel LÃpez-IbÃÃez wrote:
> On 16 February 2010 21:10, Jakub Jelinek <jakub@redhat.com> wrote:
> >
> > I think it is wrong to put two testcases into one, would be much better to
> > split it.  One issue is e.g. with __rawmemchr, as you are using target's
> > string.h, __rawmemchar might very well be defined in string.h, with
> > different properties from the declared prototype (extern "C" vs. C++,
> > throw () vs. not throw () etc.).
> > In main1 you should also fix up formatting, so it doesn't have almost 150
> > chars on a single line.
> 
> Done. OK to commit?
> 
>  Manuel.
> 
> 2010-02-16  Manuel LÃpez-IbÃÃez  <manu@gcc.gnu.org>
> 
> 	PR 36513
> testsuite/
> 	* c-c++-common/pr36513.c: New testcase.
> 	* c-c++-common/pr36513-2.c: New testcase.

OK.

Janis



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