Although this has been done
before, yours uses databases which is a great improvement (lag-wise at least). A couple of suggestions however
- all global variables and internal functions people arent supposed to directly access, make them "static stock" (it doesnt say in the article that you can use static for functions, or that you can use stock for variables, but it applies to them too)
- declare large local array variables (or small too!) with decl
- it seems the code is leaking memory by reconnecting to the database each time, discarding the potential existing database handle
- can you add support for threaded queries? dont remove the non-threaded stuff though, its nice to be able to choose
Good job implementing this on databases
__________________