diff options
author | Michael Jennings <mej@kainx.org> | 2010-08-23 17:55:09 +0000 |
---|---|---|
committer | Michael Jennings <mej@kainx.org> | 2010-08-23 17:55:09 +0000 |
commit | 8b96666c7c7dd4fc1b52fdb4b55bbc15ca0c071c (patch) | |
tree | c99e7812575034fd8d3e4c09dddc077437980b34 /src/msgs.c | |
parent | e14093a9f933fef1826d4036bf9991387020405d (diff) | |
download | libast-8b96666c7c7dd4fc1b52fdb4b55bbc15ca0c071c.tar.gz |
Revert coccinelle changes.
Using !! instead of != NULL results in significantly and unacceptably
less readable code, and I refuse to accept those changes.
Unfortunately, since they were all done at once, I have to revert the
whole thing. Oh well. :(
SVN revision: 51583
Diffstat (limited to 'src/msgs.c')
-rw-r--r-- | src/msgs.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -165,7 +165,7 @@ libast_dprintf(const char *format, ...) ASSERT_RVAL(!SPIF_PTR_ISNULL(format), (int) -1); REQUIRE_RVAL(!silent, 0); - REQUIRE_RVAL(!!libast_program_name, 0); + REQUIRE_RVAL(libast_program_name != NULL, 0); va_start(args, format); n = vfprintf(LIBAST_DEBUG_FD, format, args); va_end(args); @@ -194,7 +194,7 @@ libast_print_error(const char *fmt, ...) ASSERT(!SPIF_PTR_ISNULL(fmt)); REQUIRE(!silent); - REQUIRE(!!libast_program_name); + REQUIRE(libast_program_name != NULL); va_start(arg_ptr, fmt); fprintf(stderr, "%s: Error: ", libast_program_name); vfprintf(stderr, fmt, arg_ptr); @@ -222,7 +222,7 @@ libast_print_warning(const char *fmt, ...) ASSERT(!SPIF_PTR_ISNULL(fmt)); REQUIRE(!silent); - REQUIRE(!!libast_program_name); + REQUIRE(libast_program_name != NULL); va_start(arg_ptr, fmt); fprintf(stderr, "%s: Warning: ", libast_program_name); vfprintf(stderr, fmt, arg_ptr); @@ -249,7 +249,7 @@ libast_fatal_error(const char *fmt, ...) va_list arg_ptr; ASSERT(!SPIF_PTR_ISNULL(fmt)); - if ((!silent) && (libast_program_name)) { + if ((!silent) && (libast_program_name != NULL)) { va_start(arg_ptr, fmt); fprintf(stderr, "%s: FATAL: ", libast_program_name); vfprintf(stderr, fmt, arg_ptr); |