This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: Warnings in native MinGW32 build of GDB 7.8
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Yao Qi <yao at codesourcery dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Wed, 13 Aug 2014 18:21:33 +0300
- Subject: Re: Warnings in native MinGW32 build of GDB 7.8
- Authentication-results: sourceware.org; auth=none
- References: <83r40plpp3 dot fsf at gnu dot org> <53EAE47A dot 6030700 at codesourcery dot com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Wed, 13 Aug 2014 12:07:22 +0800
> From: Yao Qi <yao@codesourcery.com>
>
> On 08/09/2014 10:09 PM, Eli Zaretskii wrote:
> > I fixed that as below, but I wonder why no one else saw this. is this
> > because I use an ancient version of GCC?
>
> I don't see any warning in my mingw32 build. I am using
> i686-w64-mingw32-gcc 4.8.2.
Your GCC version is eons ahead of mine, and from a different distro on
top of that.
> > OK to commit the below (master and 7.8 branch), with suitable
> > ChangeLog entries?
> >
> > --- gdb/dcache.c~0 2014-07-29 15:37:42.000000000 +0300
> > +++ gdb/dcache.c 2014-08-09 16:17:31.823000000 +0300
> > @@ -18,6 +18,7 @@
> > along with this program. If not, see <http://www.gnu.org/licenses/>. */
> >
> > #include "defs.h"
> > +#include "target.h" /* for 'enum target_xfer_status' */
>
> I can see enum target_xfer_status is used in this c file. This is good
> to me.
Not sure what you mean here. Do you agree with this change? If not,
why not?
> > #include "dcache.h"
> > #include "gdbcmd.h"
> > #include <string.h>
> >
> > --- gdb/defs.h~0 2014-07-29 15:37:42.000000000 +0300
> > +++ gdb/defs.h 2014-08-09 15:33:59.666750000 +0300
> > @@ -628,6 +628,7 @@
> > #endif /* alloca not defined */
> >
> > /* Dynamic target-system-dependent parameters for GDB. */
> > +#include "frame.h" /* for 'struct frame_id' */
>
> It is unclear to me why do we need this include?
Because 'struct frame_id' is otherwise not defined, and I get warnings
like this one:
In file included from defs.h:631,
from gdb.c:19:
gdbarch.h:429: warning: parameter has incomplete type
gdbarch.h:430: warning: parameter has incomplete type
> > --- gdb/target-dcache.c~0 2014-06-11 19:34:41.000000000 +0300
> > +++ gdb/target-dcache.c 2014-08-09 16:17:42.244875000 +0300
> > @@ -16,6 +16,7 @@
> > along with this program. If not, see <http://www.gnu.org/licenses/>. */
> >
> > #include "defs.h"
> > +#include "target.h" /* for 'enum target_xfer_status' */
>
> enum target_xfer_status isn't used in target-dcache.c. Do we really
> need this?
It is used in dcache.h which target-dcache.c includes:
In file included from target-dcache.h:21,
from target-dcache.c:19:
dcache.h:42: warning: parameter has incomplete type
Thanks for reviewing the patch.