You don't need this line, as the same thing happens in the for loop below. Could then also pull the declaration into the for (although they get a little long this way).
```
for(DiagMap::iterator values_it = values_->begin(); values_it != values_->end(); ++values_it) 
```