[PATCH RFA] testsuite/gdb.c++/templates.cc printf() patch
Fernando Nasser
fnasser@redhat.com
Mon Jan 29 20:14:00 GMT 2001
Kevin Buettner wrote:
>
> On Jan 29, 6:44pm, Michael Elizabeth Chastain wrote:
>
> > Hmmm. Why does gdb.c++/templates.cc have a definition of printf()
> > at all?
>
> Good question. I didn't see any reason for the printf() definition
> either at the time I investigated this problem. But I figured that
> the more conservative course of action would be to change the
> definition to not collide with gcc's internal declarations.
>
> I just spent a few minutes digging around in the Red Hat (former
> Cygnus) internal repository and didn't find anything enlightening
> either. I thought perhaps that some earlier version of the test
> used printf() in some capacity, but that does not appear to be
> the case.
>
> > I don't think it's ever called, either from templates.cc or
> > from templates.exp. I tried adding some 'write' statements to it
> > and deleting the thing altogether, and I always got the same results
> > (5 PASS, 17 FAIL, 6 XFAIL).
> >
> > I have no problem with changing the definition from "void" to "int"
> > as long as it exists though.
>
> I'll leave it to the testsuite maintainers to decide whether it would
> be better to change the printf() declarator or remove the printf()
> definition altogether.
>
Leaving it there without knowing why does not make sense.
Scrap it.
P.S.: The good thing about the CVS repository is that you can repent
later
and undo the evil:-)
--
Fernando Nasser
Red Hat Canada Ltd. E-Mail: fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario M4P 2C9
More information about the Gdb-patches
mailing list