I\'m currently working on some quite old C++ code and often find things like
int i;
i = 42;
or
Object* someObject = NULL;
someO
No. It's a matter of style.
However if he ever wanted to move the declaration out of the function it would be less editing if the declaration and initialization is seperated.
The change you propose is highly recommended! This is part of an important idiom in C++ programming, namely Resource Acquisition Is Initialization.