]> git.mxchange.org Git - simgear.git/commit
Melchior FRANZ:
authorehofman <ehofman>
Thu, 27 Jan 2005 10:42:31 +0000 (10:42 +0000)
committerehofman <ehofman>
Thu, 27 Jan 2005 10:42:31 +0000 (10:42 +0000)
commit8e284a70b7cca7d1e8e74b4cf2a54a2754ea8512
tree5f21bdb2e3f42de4b773af3511e46067319868a4
parent73f9febe068910b1fcf219c8322bd54bd6b4e1d3
Melchior FRANZ:

Trying to find the bug in tower.cxx (that crashes fgfs quite frequently
for me!), I'm playing with valgrind again. Until I'm in the ATC subsystem
there will be some other bugs and nitpicking along the way.

valgrind doesn't like that imgage->tmp is once allocated with new and
once with new[], sometimes with malloc() (via map), and sometimes freed
with delete (not delete[]!) and sometimes with free(). With simple types
such as GLubyte this shouldn't really make a difference, but anyway.

Also, I promised that I'd send patches for "if (foo) delete foo;" as
I'm making other changes to concerned files. texture.cxx is one with a
few occurrences thereof. (Remember: C++ explicitly allows to delete
null-pointers, so this check is redundant, and hence not tolerated in
other projects, such as KDE. Doesn't have to impress us, of course.  :-)

Also, fixes 4 signed/unsigned warnings (gcc 3.3.4)
simgear/screen/texture.cxx