BIND 10 master, updated. 9500689fabae67565bee456d33953ae7b139a209 [master] for log4cplus unused variable issue

BIND 10 source code commits bind10-changes at lists.isc.org
Tue May 31 18:05:22 UTC 2011


The branch, master has been updated
       via  9500689fabae67565bee456d33953ae7b139a209 (commit)
      from  db96ce78efca85a1953fcb14d4ba24b7989044cc (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 9500689fabae67565bee456d33953ae7b139a209
Author: Jeremy C. Reed <jreed at ISC.org>
Date:   Tue May 31 13:03:54 2011 -0500

    [master] for log4cplus unused variable issue
    
    add comment
    change from unused-parameter to unused-variable
    remove additional change as was not needed.
    
    discussed on jabber
    and I tested directly with clang on Mac OS X (where problem
    was seen).

-----------------------------------------------------------------------

Summary of changes:
 src/lib/log/tests/Makefile.am |    8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/lib/log/tests/Makefile.am b/src/lib/log/tests/Makefile.am
index 2b9b245..c9f389c 100644
--- a/src/lib/log/tests/Makefile.am
+++ b/src/lib/log/tests/Makefile.am
@@ -29,7 +29,9 @@ run_unittests_LDFLAGS  = $(AM_LDFLAGS)  $(GTEST_LDFLAGS)
 
 run_unittests_CXXFLAGS = $(AM_CXXFLAGS)
 if USE_CLANGPP
-run_unittests_CXXFLAGS += -Wno-error=unused-parameter
+# This is to workaround unused variables tcout and tcerr in
+# log4cplus's streams.h.
+run_unittests_CXXFLAGS += -Wno-unused-variable
 endif
 run_unittests_LDADD  = $(GTEST_LDADD)
 run_unittests_LDADD += $(top_builddir)/src/lib/log/liblog.la
@@ -39,10 +41,6 @@ endif
 TESTS += logger_support_test
 logger_support_test_SOURCES  = logger_support_test.cc
 logger_support_test_CPPFLAGS = $(AM_CPPFLAGS)
-logger_support_test_CXXFLAGS = $(AM_CXXFLAGS)
-if USE_CLANGPP
-logger_support_test_CXXFLAGS += -Wno-error=unused-parameter
-endif
 logger_support_test_LDFLAGS  = $(AM_LDFLAGS)
 logger_support_test_LDADD    = $(top_builddir)/src/lib/log/liblog.la
 




More information about the bind10-changes mailing list