+#define APR_ARRAY_PUSH(arr, type, item) \
A macro that is intended to mimic the behavior of a function call should always be named in the same way as function calls, not with all-uppercase names. Stuff like the above makes the interface entirely dependent on the implementation, and makes the code harder to read for no reason.
....Roy
