502
|
|
|
Matthew Fuller |
8 years ago
|
 |
|
497
|
|
|
Matthew Fuller |
9 years ago
|
 |
|
496
|
|
|
Matthew Fuller |
9 years ago
|
 |
|
495
|
|
Various broad cleanups. Probably more uncaught cases of everything still lurking to be found.
- Consistently use C99 bool type and true/false values for boolean vars, struct elements, function arguments, return values, and constants. This replaces inconsistent uses of int's and shorts, with 1/0 and TRUE/FALSE #define values, and a number of uses of X11 Bool type and True/False values (in all sorts of mis-combinations). Bool and True/False are still retained for places where they're used with Xlib, as that's the right type there.
- Some minor renaming of vars and struct elements in that process.
- Various header file cleanups. Include where used, but don't make them broader than necessary (e.g., #including in a .h because you need it in the .c, in most cases). When something's in ctwm.h, we don't need it elsewhere. GC unused includes, macros, and vars. extern specifier isn't needed or useful on function prototypes. Guard everything with the idempotency guards.
- Various malloc() and friends cleanups. Don't even pretend to support systems where we have to prototype them ourselves. Don't cast malloc() returns and free() args. Use calloc() where appropriate.
- Misc pre-c89 code cleanups. NULL is a pointer value, 0 (or casts of 0) aren't. Signal handlers return void.
- Other minor changes found in the process.
|
Matthew Fuller |
9 years ago
|
 |
|
492
|
|
|
Matthew Fuller |
9 years ago
|
 |
|
482
|
|
|
Matthew Fuller |
9 years ago
|
 |
|
481
|
|
|
Matthew Fuller |
9 years ago
|
 |
|
480
|
|
|
Matthew Fuller |
9 years ago
|
 |
|