diff options
author | Martin Schwenke <martin@meltin.net> | 2014-07-31 11:03:59 +1000 |
---|---|---|
committer | Amitay Isaacs <amitay@samba.org> | 2014-08-20 16:54:31 +0200 |
commit | 47e7440be9ab422b3b2544c0b071fb8717a7a915 (patch) | |
tree | 55f40ebf504c6797e8ede790ec7cfeff1b4ac048 /lib/replace/test | |
parent | 953c88395bb9a408ebfb65329aecee6ee1a43fdc (diff) | |
download | samba-47e7440be9ab422b3b2544c0b071fb8717a7a915.tar.gz samba-47e7440be9ab422b3b2544c0b071fb8717a7a915.tar.xz samba-47e7440be9ab422b3b2544c0b071fb8717a7a915.zip |
replace: Remove unused item returned by FAILED()
The (return) value of FAILED() is a constant 1. However, it is never
used, so the compiler complains when run with -Wall:
lib/replace/test/os2_delete.c: In function ‘cleanup’:
lib/replace/test/os2_delete.c:39:163: warning: right-hand operand of comma expression has no effect [-Wunused-value]
FAILED("system");
So just get remove the ", 1" since it is the bit that does nothing and
is never used.
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Autobuild-User(master): Amitay Isaacs <amitay@samba.org>
Autobuild-Date(master): Wed Aug 20 16:54:31 CEST 2014 on sn-devel-104
Diffstat (limited to 'lib/replace/test')
-rw-r--r-- | lib/replace/test/os2_delete.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/replace/test/os2_delete.c b/lib/replace/test/os2_delete.c index a11ed3b370..9e2115bb81 100644 --- a/lib/replace/test/os2_delete.c +++ b/lib/replace/test/os2_delete.c @@ -22,7 +22,7 @@ static int test_readdir_os2_delete_ret; -#define FAILED(d) (printf("failure: readdir [\nFailed for %s - %d = %s\n]\n", d, errno, strerror(errno)), test_readdir_os2_delete_ret = 1, 1) +#define FAILED(d) (printf("failure: readdir [\nFailed for %s - %d = %s\n]\n", d, errno, strerror(errno)), test_readdir_os2_delete_ret = 1) #ifndef MIN #define MIN(a,b) ((a)<(b)?(a):(b)) |