On Aug 14, 2005, at 11:08 PM, Garrett Rooney wrote:
Rian Hunter wrote:
This patch looks good but I have some questions. You seem to use
the returned pointers from apr_array_push without checking if
they are NULL. Even in apr_array_push, apr_palloc is used without
checking for NULL even
Rian Hunter wrote:
This patch looks good but I have some questions. You seem to use the
returned pointers from apr_array_push without checking if they are
NULL. Even in apr_array_push, apr_palloc is used without checking for
NULL even though apr_palloc can definitely return NULL.
Because o
This patch looks good but I have some questions. You seem to use the
returned pointers from apr_array_push without checking if they are
NULL. Even in apr_array_push, apr_palloc is used without checking for
NULL even though apr_palloc can definitely return NULL.
Because of that, I'm not sure
Garrett Rooney wrote:
Rian Hunter wrote:
Ah I didn't even realize the key allocation, I'll fix that. Thanks!
The reason I don't use an apr_array_t or similar is that I thought
that the number of elements in that type has to be fixed and can't be
automatically extended and allocated on the