From: Steve Dickson <SteveD@redhat.com>
To: Steve Dickson <steved@redhat.com>
Cc: Linux NFS Mailing list <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH 00/24] Turn on the -Wextra compile flag (Take 2)
Date: Mon, 09 Aug 2010 10:00:50 -0400 [thread overview]
Message-ID: <4C600A12.1070103@RedHat.com> (raw)
In-Reply-To: <1280934740-11366-1-git-send-email-steved@redhat.com>
On 08/04/2010 11:11 AM, Steve Dickson wrote:
> To catch potential during compile time, the -Wextra compile flag
> was added to the CFLAGS. The raise the bar like this, with regard to
> warnings, will keep the code cleaner in the long run.
>
>
> Steve Dickson (24):
> Enable extra compile warnings (-Wextra) by default.
> Removed warnings from rpcdispatch.c
> Remove warnings from svc_socket.c
> emove warnings from cacheio.c
> Remove warnings from nfs_mntent.c
> Removed warnings from conffile.c
> Removed warnings from fsloc.c
> Removed warnings from cache.c
> Removed warnings from nfssvc.c
> Removed warnings from nfsstat.c
> Removed warnings from atomicio.c
> Removed warnings from gssd.c
> Removed warnings from gssd_main_loop.c
> Removed warnings from krb5_util.c
> Removed warnings from gssd_proc.c
> Removed warnings from svcgssd.c
> Removed warnings from nfsmount.c
> Removed warnings from nfs4mount.c
> Removed warnings from network.c
> Removed warnings from svcgssd_proc.c
> Removed warnings from configfile.c
> Removed warnings from idmapd.c
> Removed warnings from mountd.c
> Remove warnings from nfsctl.c
>
> configure.ac | 2 +-
> support/export/nfsctl.c | 2 +-
> support/include/nfslib.h | 7 +++++--
> support/nfs/cacheio.c | 2 +-
> support/nfs/conffile.c | 8 ++++----
> support/nfs/nfs_mntent.c | 2 +-
> support/nfs/rpcdispatch.c | 4 ++--
> support/nfs/rpcmisc.c | 2 +-
> support/nfs/svc_socket.c | 6 +++---
> utils/gssd/gssd.c | 2 +-
> utils/gssd/gssd_main_loop.c | 2 ++
> utils/gssd/gssd_proc.c | 8 ++++----
> utils/gssd/krb5_util.c | 29 +++++++++++++++++++++--------
> utils/gssd/krb5_util.h | 2 +-
> utils/gssd/svcgssd.c | 2 +-
> utils/gssd/svcgssd_proc.c | 12 +++---------
> utils/idmapd/atomicio.c | 3 ++-
> utils/idmapd/idmapd.c | 36 ++++++++++++++++++++++++++----------
> utils/mount/configfile.c | 6 ++++--
> utils/mount/network.c | 3 ++-
> utils/mount/nfs4mount.c | 2 +-
> utils/mount/nfsmount.c | 8 ++++++--
> utils/mountd/cache.c | 20 ++++++++++----------
> utils/mountd/fsloc.c | 2 +-
> utils/mountd/fsloc.h | 2 +-
> utils/mountd/mountd.c | 14 ++++++++------
> utils/nfsd/nfssvc.c | 6 +++---
> utils/nfsstat/nfsstat.c | 6 +++---
> 28 files changed, 119 insertions(+), 81 deletions(-)
>
Committed...
steved.
prev parent reply other threads:[~2010-08-09 14:00 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-04 15:11 [PATCH 00/24] Turn on the -Wextra compile flag (Take 2) Steve Dickson
2010-08-04 15:11 ` [PATCH 01/24] Enable extra compile warnings (-Wextra) by default Steve Dickson
2010-08-04 15:11 ` [PATCH 02/24] Removed warnings from rpcdispatch.c Steve Dickson
2010-08-04 15:11 ` [PATCH 03/24] Remove warnings from svc_socket.c Steve Dickson
2010-08-04 15:12 ` [PATCH 04/24] emove warnings from cacheio.c Steve Dickson
2010-08-04 15:12 ` [PATCH 05/24] Remove warnings from nfs_mntent.c Steve Dickson
2010-08-04 15:12 ` [PATCH 06/24] Removed warnings from conffile.c Steve Dickson
2010-08-04 15:12 ` [PATCH 07/24] Removed warnings from fsloc.c Steve Dickson
2010-08-04 15:12 ` [PATCH 08/24] Removed warnings from cache.c Steve Dickson
2010-08-04 15:12 ` [PATCH 09/24] Removed warnings from nfssvc.c Steve Dickson
2010-08-04 15:12 ` [PATCH 10/24] Removed warnings from nfsstat.c Steve Dickson
2010-08-04 15:12 ` [PATCH 11/24] Removed warnings from atomicio.c Steve Dickson
2010-08-04 15:12 ` [PATCH 12/24] Removed warnings from gssd.c Steve Dickson
2010-08-04 15:12 ` [PATCH 13/24] Removed warnings from gssd_main_loop.c Steve Dickson
2010-08-04 15:12 ` [PATCH 14/24] Removed warnings from krb5_util.c Steve Dickson
2010-08-04 15:12 ` [PATCH 15/24] Removed warnings from gssd_proc.c Steve Dickson
2010-08-04 15:12 ` [PATCH 16/24] Removed warnings from svcgssd.c Steve Dickson
2010-08-04 15:12 ` [PATCH 17/24] Removed warnings from nfsmount.c Steve Dickson
2010-08-04 15:12 ` [PATCH 18/24] Removed warnings from nfs4mount.c Steve Dickson
2010-08-04 15:12 ` [PATCH 19/24] Removed warnings from network.c Steve Dickson
2010-08-04 15:12 ` [PATCH 20/24] Removed warnings from svcgssd_proc.c Steve Dickson
2010-08-04 15:12 ` [PATCH 21/24] Removed warnings from configfile.c Steve Dickson
2010-08-04 15:12 ` [PATCH 22/24] Removed warnings from idmapd.c Steve Dickson
2010-08-04 15:12 ` [PATCH 23/24] Removed warnings from mountd.c Steve Dickson
2010-08-04 15:12 ` [PATCH 24/24] Remove warnings from nfsctl.c Steve Dickson
2010-08-09 14:00 ` Steve Dickson [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=4C600A12.1070103@RedHat.com \
--to=steved@redhat.com \
--cc=linux-nfs@vger.kernel.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;
as well as URLs for NNTP newsgroup(s).