From: Paul Brook <paul@codesourcery.com>
To: Dirk Behme <dirk.behme@googlemail.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] Fix warnings in mips_r4k.c
Date: Mon, 31 Jul 2006 14:33:14 +0100 [thread overview]
Message-ID: <200607311433.15162.paul@codesourcery.com> (raw)
In-Reply-To: <44CE032D.7050709@gmail.com>
> >>hw/mips_r4k.c:256: warning: pointer targets in passing
> >>argument 1 of ‘strcpy’ differ in signedness
> >
> > Compile with -Wno-pointer-sign. gcc4 isn't really supported anyway.
>
> Yes, I know ;) And yes, I know there are technical reasons
> for not supporting gcc4.
>
> But do you wan't to say that at places, where because of
> typos or lazy programming and thus no technical reason for
> it (e.g. passing signed where calling function wants
> unsigned) it is better to disable warning with
> -Wno-pointer-sign instead of fixing it?
I'm saying that this warning occurs in sufficiently many places, many of which
are questionable, that it's best to just turn it off.
The entry change is maybe acceptable. The strcpy change just uglifies the
code.
Paul
prev parent reply other threads:[~2006-07-31 13:33 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-31 10:20 [Qemu-devel] [PATCH] Fix warnings in mips_r4k.c Dirk Behme
2006-07-31 12:39 ` Paul Brook
2006-07-31 13:18 ` Dirk Behme
2006-07-31 13:33 ` Paul Brook [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200607311433.15162.paul@codesourcery.com \
--to=paul@codesourcery.com \
--cc=dirk.behme@googlemail.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.