From: Pedro Alves <palves@redhat.com>
To: gdb-patches@sourceware.org
Subject: [PATCH 00/14] -Wmissing-prototypes: Intro
Date: Wed, 29 Feb 2012 16:17:00 -0000 [thread overview]
Message-ID: <20120229161628.23918.51354.stgit@hit-nxdomain.opendns.com> (raw)
It's that time of the year again...
Back in Feb 2008, and then on Feb 2009 again, I cleaned up the tree to
make it buildable with -Wmissing-prototypes on
x86_64-unknown-linux-gnu (--enable-targets=all, I think), but I ended
up not adding -Wmissing-prototypes to the default warnings set. Since
then, another set of -Wmissing-prototypes problems cropped up.
So this series fixes all I could see, and then proposes finally
enabling -Wmissing-prototypes by default, to make sure I won't go
through this again in Feb 2013...
This warning is quite useful as it catches these kinds of issues:
- modules (.c files) that don't include their own header, where
there's potential to change some function's prototype in the .c
file, and forget to adjust it in the .h file, and then you end up
scratching your head wondering why is GDB behaving funny or
crashing (because clients of your function hadn't been adjusted to
the new prototype, because they were still being compiled with the
old prototype in the header).
- It's harder to end up with extern functions that should be static,
because the compiler will complain that it hasn't seen a prototype
for the function. In most cases, the fix for the warning it to
make the function static.
- Helps get rid of unused functions. The above point means that as
soon as you make the function static, if there are no callers to
the function, the compiler will complain about an unused static
function, so you end up deleting the dead code.
There are examples of all these in the series.
Tested on x86_64 Fedora 16.
---
Pedro Alves (14):
-Wmissing-prototypes: Ada.
-Wmissing-prototypes: proc-service.
-Wmissing-prototypes: Garbage collect inferior.c:delete_threads_of_inferior.
-Wmissing-prototypes: Python
-Wmissing-prototypes: cleanup encode_actions hack.
-Wmissing-prototypes: TUI.
-Wmissing-prototypes: inline-frame.c.
-Wmissing-prototypes: The find_and_open_source hack.
-Wmissing-prototypes: jit-reader.in (plugin_is_GPL_compatible).
-Wmissing-prototypes: common/signals.c.
-Wmissing-prototypes: observer.c's testsuite helpers.
-Wmissing-prototypes: hook linux_has_shared_address_space.
-Wmissing-prototypes: All others.
-Wmissing-prototypes: Build with -Wmissing-prototypes by default.
next reply other threads:[~2012-02-29 16:16 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-29 16:17 Pedro Alves [this message]
2012-02-29 16:17 ` [PATCH 03/14] -Wmissing-prototypes: Garbage collect inferior.c:delete_threads_of_inferior Pedro Alves
2012-02-29 16:17 ` [PATCH 02/14] -Wmissing-prototypes: proc-service Pedro Alves
2012-02-29 16:17 ` [PATCH 01/14] -Wmissing-prototypes: Ada Pedro Alves
2012-02-29 17:19 ` Joel Brobecker
2012-02-29 16:18 ` [PATCH 05/14] -Wmissing-prototypes: The encode_actions hack Pedro Alves
2012-02-29 16:18 ` [PATCH 04/14] -Wmissing-prototypes: Python Pedro Alves
2012-02-29 18:09 ` Tom Tromey
2012-02-29 16:18 ` [PATCH 06/14] -Wmissing-prototypes: The TUI Pedro Alves
2012-02-29 16:19 ` [PATCH 07/14] -Wmissing-prototypes: inline-frame.c Pedro Alves
2012-02-29 16:20 ` [PATCH 10/14] -Wmissing-prototypes: common/signals.c Pedro Alves
2012-02-29 16:20 ` [PATCH 08/14] -Wmissing-prototypes: The find_and_open_source hack Pedro Alves
2012-02-29 16:21 ` [PATCH 11/14] -Wmissing-prototypes: observer.c's testsuite helpers Pedro Alves
2012-02-29 16:44 ` [PATCH 14/14] -Wmissing-prototypes: Build with -Wmissing-prototypes by default Pedro Alves
2012-02-29 19:11 ` Tom Tromey
2012-02-29 22:41 ` Joel Brobecker
2012-02-29 17:07 ` [PATCH 13/14] -Wmissing-prototypes: All others Pedro Alves
2012-02-29 17:08 ` [PATCH 09/14] -Wmissing-prototypes: jit-reader.in (plugin_is_GPL_compatible) Pedro Alves
2012-02-29 17:18 ` Pedro Alves
2012-03-01 21:25 ` C version of jit-reader.h:plugin_is_GPL_compatible broken Pedro Alves
2012-02-29 17:12 ` [PATCH 12/14] -Wmissing-prototypes: Hook linux_has_shared_address_space Pedro Alves
2012-02-29 17:27 ` [PATCH 00/14] -Wmissing-prototypes: Intro Joel Brobecker
2012-02-29 17:42 ` Joel Brobecker
2012-03-01 21:19 ` Pedro Alves
2012-03-02 12:03 ` Pedro Alves
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=20120229161628.23918.51354.stgit@hit-nxdomain.opendns.com \
--to=palves@redhat.com \
--cc=gdb-patches@sourceware.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox