Whamcloud - gitweb
remove useless if-before-free tests
authorJim Meyering <jim@meyering.net>
Mon, 23 Feb 2009 17:07:50 +0000 (18:07 +0100)
committerTheodore Ts'o <tytso@mit.edu>
Mon, 9 Mar 2009 00:37:12 +0000 (20:37 -0400)
commit45e338f5332a54295893dba2e32cc093d1316f60
treef20a569662c01e9330b33ea9fd8baa83c7601473
parenta6d4aa147415acf875300176c5ca98ca72594bec
remove useless if-before-free tests

In case you're wondering about whether this change is safe from a
portability standpoint, fear not.  This has been beaten to death
in other forums.  Here are a few threads:

  http://thread.gmane.org/gmane.comp.version-control.git/74187
  http://thread.gmane.org/gmane.comp.lib.gnulib.bugs/12712
  http://thread.gmane.org/gmane.emacs.devel/98144
  http://thread.gmane.org/gmane.comp.lib.glibc.alpha/13092

There has been debate about whether it's a good idea from a
performance standpoint, too, but imho you'll have a hard time
finding an instance where this sort of change induces a
measurable performance penalty.  If you do, please let me know.

Signed-off-by: Jim Meyering <meyering@redhat.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
31 files changed:
debugfs/htree.c
debugfs/icheck.c
e2fsck/argv_parse.c
e2fsck/dirinfo.c
e2fsck/profile.c
e2fsck/rehash.c
intl/bindtextdom.c
intl/loadmsgcat.c
intl/localcharset.c
intl/printf-parse.c
intl/vasnprintf.c
lib/blkid/cache.c
lib/blkid/dev.c
lib/blkid/devno.c
lib/blkid/probe.c
lib/blkid/resolve.c
lib/blkid/save.c
lib/blkid/tag.c
lib/ext2fs/dosio.c
lib/ext2fs/extent.c
lib/ext2fs/initialize.c
lib/ext2fs/nt_io.c
lib/ss/execute_cmd.c
misc/badblocks.c
misc/blkid.c
misc/e2initrd_helper.c
misc/fsck.c
misc/mke2fs.c
misc/tune2fs.c
resize/resize2fs.c
util/subst.c