diff options
author | Jacob Champion <jchampion@apache.org> | 2016-11-10 21:53:21 +0100 |
---|---|---|
committer | Jacob Champion <jchampion@apache.org> | 2016-11-10 21:53:21 +0100 |
commit | 091f96ee10e3963faacd258c3f64417c7ad21e1c (patch) | |
tree | ffd8ec24d41ec9a54cf20c7580aaa0ce3ce673fd /ABOUT_APACHE | |
parent | heh... bring memcache up to redis :) (diff) | |
download | apache2-091f96ee10e3963faacd258c3f64417c7ad21e1c.tar.xz apache2-091f96ee10e3963faacd258c3f64417c7ad21e1c.zip |
Remove unnecessary apr_table_do() function casts
Function casts can cause hard-to-debug corruption issues if a
declaration is accidentally changed to be incompatible. Luckily, most of
the function casts for apr_table_do() calls are unnecessary. Remove
them, and adjust the signatures for helpers that weren't taking void* as
the first argument.
The remaining helper that requires a cast is http_filter.c's
form_header_field(), which is probably where many of these casts were
copy-pasted from. I have left it as-is: it has other direct callers
besides apr_table_do(), and it's already documented with warnings not to
change the function signature.
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1769192 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'ABOUT_APACHE')
0 files changed, 0 insertions, 0 deletions