public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Gerald Pfeifer <gerald@pfeifer.com>
To: Stephan Thomas Lavavej <stl@caltech.edu>
Cc: gcc@gcc.gnu.org, gcc-patches@gcc.gnu.org
Subject: PATCH for: Lots of suggestions for the gcc manual
Date: Fri, 03 Oct 2003 18:54:00 -0000	[thread overview]
Message-ID: <Pine.BSF.4.58.0310032050250.22892@naos.dbai.tuwien.ac.at> (raw)
In-Reply-To: <Pine.GSO.4.58.0309130347490.24873@blinky>

On Sat, 13 Sep 2003, Stephan Thomas Lavavej wrote:
> http://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#Warning%20Options

What do you think about the following patch to address several specific
observations you made?

(Concerning your general remarks on how to restructure the list of
options, I'd slightly prefer the variant someone else suggested in
a followup.  For that, however, I'm afraid we'd need to a volunteer
to summarize the suggested changes to the gcc list and, once consensus
has been reached, provide a patch.  I'd really appreciate that.)

Gerald

2003-10-03  Gerald Pfeifer  <gerald@pfeifer.com>

	* doc/invoke.texi (Warning Options): Simplify and clarify the
	descriptions of -Wnonnull and -Winit-self.

Index: doc/invoke.texi
===================================================================
RCS file: /cvs/gcc/gcc/gcc/doc/invoke.texi,v
retrieving revision 1.339
diff -u -3 -p -r1.339 invoke.texi
--- doc/invoke.texi	25 Sep 2003 01:25:52 -0000	1.339
+++ doc/invoke.texi	3 Oct 2003 18:50:38 -0000
@@ -2172,7 +2172,7 @@ Enable @option{-Wformat} plus format che

 @item -Wnonnull
 @opindex Wnonnull
-Enable warning about passing a null pointer for arguments marked as
+Warn about passing a null pointer for arguments marked as
 requiring a non-null value by the @code{nonnull} function attribute.

 @option{-Wnonnull} is included in @option{-Wall} and @option{-Wformat}.  It
@@ -2180,12 +2180,12 @@ can be disabled with the @option{-Wno-no

 @item -Winit-self @r{(C, C++, and Objective-C only)}
 @opindex Winit-self
-Enable warning about uninitialized variables which are initalized with themselves.
-Note this option can only be used with the @option{-Wuninitialized} option and
-that only works with @option{-O}.
+Warn about uninitialized variables which are initialized with themselves.
+Note this option can only be used with the @option{-Wuninitialized} option,
+which in turn only works with @option{-O1} and above.

-For an example, the following code will not warn about i being uninitialized
-without this option:
+For example, GCC will warn about @code{i} being uninitialized in the
+following snippet only when @option{-Winit-self} has been specified:
 @smallexample
 @group
 int f()

  parent reply	other threads:[~2003-10-03 18:54 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-09-13 15:00 Stephan Thomas Lavavej
2003-09-14 22:10 ` Kai Henningsen
2003-09-21 17:20   ` Gerald Pfeifer
2003-10-03 18:54 ` Gerald Pfeifer [this message]
2003-10-04 18:38   ` PATCH for: " Stephan T. Lavavej
2003-10-05 21:30     ` Gerald Pfeifer
2004-02-16  2:27       ` [PATCH] GCC Warning Options Documentation Stephan T. Lavavej
2004-02-16  3:33         ` James Morrison
2004-03-27 15:38         ` Gerald Pfeifer

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=Pine.BSF.4.58.0310032050250.22892@naos.dbai.tuwien.ac.at \
    --to=gerald@pfeifer.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gcc@gcc.gnu.org \
    --cc=stl@caltech.edu \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).