Fix typo spotted by Giulio possibly causing memory leak

[wmtan]

Fix obvious typo spotted by Giulio Eulisse. Thank you, Giulio! This typo has been in the code for ROOT6 back as far as 7_0_ROOT6_X, but was only recently introduced into the code for ROOT5 only very recently. The effect of this but is that a loop possibly invoking destructors terminates immediately, thereby possibly causing a memory leak.
I tested this fix with the unit test for the package and the entire short relval matrix, and all the tests pass.
As this is a bug fix, it should be expedited.
This fix will be identical in 7_4_X, 7_4_ROOT6_X, 7_5_X, and 7_5_ROOT5_X.

[cmsbuild]

A new Pull Request was created by @wmtan for CMSSW_7_4_X.

Fix typo spotted by Giulio possibly causing memory leak

It involves the following packages:

CommonTools/Utils

@nclopezo, @cvuosalo, @monttj, @cmsbuild, @slava77, @vadler can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[wmtan]

@ktf This is the fix for the bug you spotted. It should propagate forward to 7_4_ROOT6_X, 7_5_X, and 7_5_ROOT5_X. Let me know if you want me to explicitly request it in any of those releases.
@wddgit FYI.

[cmsbuild]

+1
Tested at: 37ceb3c
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8603/4023/summary.html

[ktf]

Current version is clearly wrong. Merging.

[ktf]

Say thanks to clang.. ;-)

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8603/4023/summary.html