running sparse over a new codebase is usually a fun way to get to know it while
finding silly bugs (in either the code or in sparse).
first patch here should have no semantic changes, just cleaups.
Created attachment 784[details]
q3-sparse-cleanups-1.patch
- many instances of s/0/NULL/
- fix a few functions like void foo() {} to be void foo(void) {}
- Com_Printf was being called in a few places with an enum for the first arg
instead of a string. pretty obvious that the enum arg should just be dropped.
Setting a QA contact on all ioquake3 bugs, even resolved ones. Sorry if you get a flood of email from this, it should only happen once. Apologies for the incovenience.
--ryan.
Created attachment 784 [details] q3-sparse-cleanups-1.patch - many instances of s/0/NULL/ - fix a few functions like void foo() {} to be void foo(void) {} - Com_Printf was being called in a few places with an enum for the first arg instead of a string. pretty obvious that the enum arg should just be dropped.Created attachment 785 [details] q3-sparse-cleanups-2.patch first one was lame. same thing, more files.