Small, general code cleanup 

Removed dead code, improved alignment, fixed warnings, and cleaned up `DumpTGA()`.
That's because `TGAHeader` takes them in short, so there would've been an error if int was used (I think in the other case it was implicitly casted). 

Should I keep it this way, or change the `TGAHeader` shorts to ints?