[Super-tux-devel] [fwd] Supertux Bugreport [SDL_Quit]

[Bill K. <nb...@so...>]

Volker Grabsch sent in this comment about using SDL_Quit, and mentioning
the unstarted 'Super Tux Adventures' game at SourceForge...

-bill!


----- Forwarded message from Volker Grabsch -----

Date: Sat, 3 Jan 2004 17:36:21 +0100
From: Volker Grabsch
Subject: Supertux Bugreport
To: Bill Kendrick <bi...@ne...>

Hi Bill,

I just downloaded and tested your game "supertux". I think it's great,
although it isn't complete yet :)

I saw a small mistake in your sources you should correct in future
releases. The "SDL_Quit" is not called at the end of your program
in each case. The reason for this is (I think) that you're calling
exit(1) without SDL_Quit and that you don't react on SIGINT.

I think, the best solution for this problem is not to call st_shutdown()
directly at the end of main(), and instead using atexit() this way:
Put directly after your SDL_Init the following code:

	atexit(SDL_Quit);

or, if you plan to put some more into your st_shutdown():

	atexit(st_shutdown);

I think, atexit() really _is_ the better solution. It simply takes
care of some types of program termination you normally don't care
about.


Well, there's also another question I have:

	http://supertux.sourceforge.net/

is a project with a very similar goal as yours. They are at the
very beginning and didn't code anything until now, but there are
many nice ideas, so I suggest you work together with them. Unless
there are deep differences between you and CharlieG, I think it
would be simply a loss when you both work independently without
having ever talked to each other. Well, that's _my_ opinion ..


Last but not least my best wishes to you for the new year 2004,
and happy coding!


Greets,

	Volker

-- 
Volker Grabsch
---<<(())>>---
\frac{\left|\vartheta_0\times\{\ell,\kappa\in\Re\}\right|}{\sqrt
[G]{-\Gamma(\alpha)\cdot\mathcal{B}^{\left[\oint\!c_\hbar\right]}}}


----- End forwarded message -----

-- 
bi...@ne...                           Got kids?  Get Tux Paint! 
http://newbreedsoftware.com/bill/       http://newbreedsoftware.com/tuxpaint/