The first thing I'd do in that case is to autoformat the indenting into something approaching sanity, even if I had no intention of making a 500 whitespace change commit just to make everything else easier.
Did you ever see history portrayed as an old man with a wise brow and pulseless heart, weighing all things in the balance of reason?
Is not rather the genius of history like an eternal, imploring maiden, full of fire, with a burning heart and flaming soul, humanly warm and humanly beautiful?
Training a telescope on one’s own belly button will only reveal lint. You like that? You go right on staring at it. I prefer looking at galaxies.
-- Sarah Hoyt
Hmmm, be careful with removing braces in the middle of the function. Unless you know why the other engineer put them there. They can be used to enforce variable scope. You can actually use them to reduce the memory usage in a thread.
Trust me ... this guy wasn't that clever
As for variable scope ... most of his stuff was global
The whole thing was re-engineered - was clearer to read, faster to operate and used far less memory (and "leaked" none) by the time we finished.
A solo developer should always write code with the next solo developer in mind. He might be very large and highly psychotic - a few diligent keystrokes today could save several indiscriminate axe-strokes tomorrow.