BIND 10 trac3112, updated. 384b365bc6895e251876cfbd440bee03076a87de [3112] Fix an uninit variable use warning
BIND 10 source code commits
bind10-changes at lists.isc.org
Thu Sep 12 14:11:39 UTC 2013
The branch, trac3112 has been updated
via 384b365bc6895e251876cfbd440bee03076a87de (commit)
from cca46d0df7a4c3008e83d84e863a9be877f42b12 (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 384b365bc6895e251876cfbd440bee03076a87de
Author: Mukund Sivaraman <muks at isc.org>
Date: Thu Sep 12 19:34:20 2013 +0530
[3112] Fix an uninit variable use warning
This apparently happens when gtest-1.5 is in use. Patch was reviewed by
Thomas Markwalder.
-----------------------------------------------------------------------
Summary of changes:
src/lib/dhcpsrv/tests/dhcp_parsers_unittest.cc | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
-----------------------------------------------------------------------
diff --git a/src/lib/dhcpsrv/tests/dhcp_parsers_unittest.cc b/src/lib/dhcpsrv/tests/dhcp_parsers_unittest.cc
index 62f901c..53e304d 100644
--- a/src/lib/dhcpsrv/tests/dhcp_parsers_unittest.cc
+++ b/src/lib/dhcpsrv/tests/dhcp_parsers_unittest.cc
@@ -714,9 +714,8 @@ public:
template<typename ContainerType, typename ValueType>
void checkValueEq(const boost::shared_ptr<ContainerType>& ref_values,
const boost::shared_ptr<ContainerType>& values) {
- ValueType param;
- ASSERT_NO_THROW(param = values->getParam("foo"));
- EXPECT_EQ(ref_values->getParam("foo"), param);
+ ASSERT_NO_THROW(values->getParam("foo"));
+ EXPECT_EQ(ref_values->getParam("foo"), values->getParam("foo"));
}
/// @brief Check that the storages of the specific type hold different
@@ -734,9 +733,8 @@ public:
template<typename ContainerType, typename ValueType>
void checkValueNeq(const boost::shared_ptr<ContainerType>& ref_values,
const boost::shared_ptr<ContainerType>& values) {
- ValueType param;
- ASSERT_NO_THROW(param = values->getParam("foo"));
- EXPECT_NE(ref_values->getParam("foo"), param);
+ ASSERT_NO_THROW(values->getParam("foo"));
+ EXPECT_NE(ref_values->getParam("foo"), values->getParam("foo"));
}
/// @brief Check that option definition storage in the context holds
More information about the bind10-changes
mailing list