All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@bergmann-dalldorf.de>
To: root@chaos.analogic.com, schwidefsky@de.ibm.com,
	linux-kernel@vger.kernel.org,
	kernel-janitor-discuss@lists.sourceforge.net
Subject: Re: [PATCH] s390 (7/13): gcc 3.3 adaptions.
Date: Mon, 24 Feb 2003 21:03:13 +0100	[thread overview]
Message-ID: <200302242005.VAA04400@post.webmailer.de> (raw)
In-Reply-To: 20030224195008$40bd@gated-at.bofh.it

Richard B. Johnson wrote:

> I think you must keep these warnings in! There are many bugs
> that these uncover uncluding loops that don't terminate correctly
> but seem to work for "most all" cases. These are the hard-to-find
> bugs that hit you six months after release.

This was my change. Obviously the warning is a good idea in general,
but I don't see the point of scrolling through hundreds of lines
with the same warning in someone else's code. I actually plan to fix
these warnings in arch/s390 and drivers/s390 as well as include/
and make the s390 kernel compile with -Werror, but the rest looks 
more like a task for the Janitors. Note that before gcc-3.3, 
-Wsign-compare has not been part of -Wall.

        Arnd <><

       reply	other threads:[~2003-02-24 19:55 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20030224195008$59ef@gated-at.bofh.it>
     [not found] ` <20030224195008$40bd@gated-at.bofh.it>
2003-02-24 20:03   ` Arnd Bergmann [this message]
2003-02-24 20:49     ` [PATCH] s390 (7/13): gcc 3.3 adaptions Richard B. Johnson
2003-02-24 18:10 Martin Schwidefsky
2003-02-24 19:44 ` Richard B. Johnson
2003-02-24 21:02   ` Linus Torvalds
2003-02-24 21:27     ` Jeff Garzik
2003-02-24 21:33     ` Richard B. Johnson
2003-02-24 21:35     ` Andreas Schwab
2003-02-24 21:53       ` Jeff Garzik
2003-02-24 22:07         ` Andreas Schwab
2003-02-24 22:21           ` Jörn Engel
2003-02-24 22:31             ` Jörn Engel
2003-02-25 12:34             ` Andreas Schwab
2003-02-24 22:39           ` Linus Torvalds
2003-02-25  0:20             ` Alan Cox
2003-02-24 23:20               ` Linus Torvalds
2003-02-25  0:40                 ` Alan Cox
2003-02-24 23:51                   ` Linus Torvalds
2003-02-25  5:24             ` Chris Wedgwood
2003-02-25 12:25               ` Jamie Lokier
2003-02-25 12:40             ` Andreas Schwab
2003-02-25 15:27               ` Linus Torvalds
2003-02-25 15:39                 ` Andreas Schwab
2003-02-25 16:02                   ` Linus Torvalds
2003-02-25 16:15                     ` Linus Torvalds
2003-02-25 16:26                     ` Andreas Schwab
2003-02-25 16:17                 ` Richard B. Johnson
2003-02-24 22:28       ` Linus Torvalds
2003-02-24 22:39       ` Jakub Jelinek
2003-02-24 23:52         ` John Alvord
2003-02-24 21:38     ` Adrian Bunk
2003-03-01  3:12     ` Richard Henderson

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=200302242005.VAA04400@post.webmailer.de \
    --to=arnd@bergmann-dalldorf.de \
    --cc=kernel-janitor-discuss@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=root@chaos.analogic.com \
    --cc=schwidefsky@de.ibm.com \
    /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.