BIND 10 #3312: Fix new cppcheck reports

[BIND 10 Development]

#3312: Fix new cppcheck reports
-------------------------------------+-------------------------------------
            Reporter:  muks          |                        Owner:  muks
                Type:  defect        |                       Status:
            Priority:  medium        |  closed
           Component:  Unclassified  |                    Milestone:
            Keywords:                |  bind10-1.2-release-freeze
           Sensitive:  0             |                   Resolution:  fixed
         Sub-Project:  DNS           |                 CVSS Scoring:
Estimated Difficulty:  0             |              Defect Severity:  N/A
         Total Hours:  0             |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
Changes (by muks):

 * status:  reviewing => closed
 * resolution:   => fixed


Comment:

 Merged to `master` branch in commit
 `aafa7f654275feaec7258b8434dd845f8400da8a`:
 {{{
 * 6f07edd (origin/trac3312, trac3312) [3312] Remove redundant variable
 (cppcheck report)
 * 5c806a2 [3312] Avoid C++ language parser warning of uninitialized
 variable
 * bbee45a [3312] Pass variables by reference (cppcheck report, etc.)
 * c415976 [3312] Reduce scope of variables (cppcheck report)
 }}}

 Resolving as fixed. Thank you for the review Stephen.

-- 
Ticket URL: <http://bind10.isc.org/ticket/3312#comment:5>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development