All of lore.kernel.org
 help / color / mirror / Atom feed
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

      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.