[PATCH] replace: Remove unused item returned by FAILED()

Martin Schwenke martin at meltin.net
Tue Aug 12 03:29:36 MDT 2014


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.

peace & happiness,
martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-replace-Remove-unused-item-returned-by-FAILED.patch
Type: text/x-patch
Size: 1446 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20140812/ebc3f5fd/attachment.bin>


More information about the samba-technical mailing list