tools/sc_main old and not required

I d-loaded LuaCheia (5-5.0.1a5) for fun and curiosity.  There's some 
observations:


- binary_packager/

Where's the 'gen_install_script.lua' file for Inno Setup 3?  Has 
someone actually used it?


- tools/sc_main/*

Seems like an awfully old remnant of pre-Hamster build system, or parts 
of it.
I'm pretty sure none of this is actually used, could you please remove 
it?

(tools/Makefile.am affected too, remove from "subdirs")


- doc/sdl.x

 >Could REALLY do with an OO API.

SDL module has, but the wrapper does it.  Not sure if LuaCheia only has 
the C/Lua module, and not the wrapper at all?  Or if the code's simply 
so old that OO aspect is not there?

 >Why is this module name uppercase when all the others aren't?

SDL has upper case. Simple Drawing Library (or .. anyhow it's an 
acronym)
Well, since others (pcre) are acronyms, too, I see no reason to treat 
SDL differently.  Your choice. :)


- LICENSE.txt

Name's Kauppi, with an 'a'. :)  Thanks for mentioning.


- src/gluahost

Come to think of it, you could also compile 'sdl_module.cpp' into a 
normal (non-gluahost) module, if you'd like.  This would mean:
	- getting rid of the gluahost 'proxy' for loading gluax-bound modules
	- putting gluax.* files with the SDL module, and compiling them there
	- #define GLUA_STATIC is the key, it will tie to Lua core stuff 
directly instead of through the gluax function table.

If you're interested in this solution, I could help you get it working. 
  Hopefully, with the latest SDL module?

-ak



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click