This is the mail archive of the gdb-patches@sourceware.org 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: [cosmetic-patch] missing declaration in monitor.c

From: Daniel Jacobowitz <drow at false dot org>
To: Denis PILAT <denis dot pilat at st dot com>
Cc: gdb-patches at sources dot redhat dot com
Date: Mon, 17 Jul 2006 09:43:58 -0400
Subject: Re: [cosmetic-patch] missing declaration in monitor.c
References: <44BB751B.10608@st.com> <20060717123228.GA8764@nevyn.them.org> <44BB877C.1050003@st.com>

On Mon, Jul 17, 2006 at 02:50:05PM +0200, Denis PILAT wrote:
> No I don't need them.
> While reading the code I found that they were missing because all other 
> static function are declared.
> I'm wondering why you remove only these 2 one ? Most of static functions 
> are not used before beeing defined.

I removed those two because I was changing them - the old ones were
wrong.  It's a bit of work to make sure that a prototype isn't
necessary.

-- 
Daniel Jacobowitz
CodeSourcery

Follow-Ups:
- Re: [cosmetic-patch] missing declaration in monitor.c
  - From: Mark Kettenis

References:
- [cosmetic-patch] missing declaration in monitor.c
  - From: Denis PILAT
- Re: [cosmetic-patch] missing declaration in monitor.c
  - From: Daniel Jacobowitz
- Re: [cosmetic-patch] missing declaration in monitor.c
  - From: Denis PILAT

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