This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB 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: [PATCH] gdbreplay.c


On Tue, Jul 09, 2002 at 10:25:41AM -0700, David O'Brien wrote:
> I'd like to commit this change.
> 
> This seems more proper for the includes includes, and the staticization
> is needed to quiet GCC at high warning levels.

I'm not sure if you're asking or telling since it sounds like you've
committed the matching patch to the 5.2 branch?

In any case, no objection, so please commit it.  But when does GCC warn
about a function not being static?  Doesn't make a lot of sense to me;
it doesn't know that you won't link with -Bsymbolic and use DSO's, for
instance.

-- 
Daniel Jacobowitz                           Carnegie Mellon University
MontaVista Software                         Debian GNU/Linux Developer


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