configure: Stop avoiding -Wformat-zero-length.
authorBen Pfaff <blp@nicira.com>
Sun, 7 Jun 2015 16:48:14 +0000 (09:48 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 10 Jun 2015 16:19:39 +0000 (09:19 -0700)
commit40e7cf5607052d3b4fa09fd433fa630352c115b6
tree4480696260487d892a041a299f5d6820c7556f2d
parentbdd7ecf5bfc4a255872aa60057b3b96f72b47d8a
configure: Stop avoiding -Wformat-zero-length.

Debian likes to enable -Wformat-zero-length, even over our code trying to
disable it.  It isn't too hard to make our code warning-free against this
option, so this commit both stops disabling it and fixes the warnings.

The first fix is to change set_subprogram_name() to take a plain string
instead of a format string, and to adjust its few callers.  This fixes one
warning since one of those callers passed in an empty string.

The second fix is to remove a test for ovs_scan() against an empty string.
I couldn't find a way to avoid a warning for this test, and it isn't too
valuable in any case.

This allows us to drop filtering for -Wformat from the Debian rules file,
so this commit removes it.

Signed-off-by: Ben Pfaff <blp@nicira.com>
configure.ac
debian/rules
lib/ovs-thread.c
lib/util.c
lib/util.h
tests/test-util.c