On Thu, Feb 01, 2018 at 03:01:41PM -0000, yla...@apache.org wrote:
> Author: ylavic
> Date: Thu Feb 1 15:01:40 2018
> New Revision: 1822879
> URL: http://svn.apache.org/viewvc?rev=1822879&view=rev
> mod_proxy: follow up to r1822849 and r1822878.
> Does r1822878's "static" APR_RETRIEVE_OPTIONAL_FN work if, say, mod_proxy is
> builtin but mod_http2 isn't?
I'd guess not!
> Not worth taking the risk here since it's not a fast path...
> Note: if this is an issue, I'm afraid it applies elsewhere too.
The only places I found with optional fn declared static within a
function were both pointlessly static and safe, fixed in r1822931. It's
funny since we have a specific hook for retrieving optional functions at
the right time, but it's almost never used. Most modules do it in
post_config which is safe.