BIND 10 #2507: Cppcheck warnings for version 1.57

[BIND 10 Development]

#2507: Cppcheck warnings for version 1.57
-------------------------------------+-------------------------------------
            Reporter:  vorner        |                        Owner:
                Type:  defect        |                       Status:  new
            Priority:  medium        |                    Milestone:  Next-
           Component:  Unclassified  |  Sprint-Proposed
           Sensitive:  0             |                     Keywords:
         Sub-Project:  Core          |              Defect Severity:  N/A
Estimated Difficulty:  0             |  Feature Depending on Ticket:
         Total Hours:  0             |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
 As usual, new version of cppcheck has new errors. After filtering out the
 „Skipping configuration“ ones (which are non-fatal and ignored on
 purpose),
 this is the current output. Putting it to next-sprint-proposed, as I
 already
 missed a problem because of this.

 {{{
 src/bin/dhcp6/tests/config_parser_unittest.cc:105: check_fail: Variable
 'first' is assigned a value that is never used. (style,unreadVariable)
 src/lib/cc/data.cc:313: check_fail: Variable 'c' is reassigned a value
 before the old one has been used. (performance,redundantAssignment)
 src/lib/cc/data.cc:400: check_fail: Variable 'i' is reassigned a value
 before the old one has been used. (performance,redundantAssignment)
 src/lib/datasrc/tests/memory/rdata_serialization_unittest.cc:487:
 check_fail: Variable 'current' is reassigned a value before the old one
 has been used. (performance,redundantAssignment)
 src/lib/datasrc/tests/memory/rdata_serialization_unittest.cc:412:
 check_fail: Variable 'current' is assigned a value that is never used.
 (style,unreadVariable)
 src/lib/datasrc/tests/memory/rdata_serialization_unittest.cc:487:
 check_fail: Variable 'current' is assigned a value that is never used.
 (style,unreadVariable)
 src/lib/dhcp/tests/libdhcp++_unittest.cc:97: check_fail: Variable
 'factory' is reassigned a value before the old one has been used.
 (performance,redundantAssignment)
 }}}

-- 
Ticket URL: <http://bind10.isc.org/ticket/2507>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development