On Mon, Jun 27, 2022 at 8:08 PM Ivan Zhakov <i...@visualsvn.com> wrote: > > On Mon, 27 Jun 2022 at 21:01, <yla...@apache.org> wrote: >> >> +#ifdef WIN32 >> +typedef apr_status_t (__stdcall *encdec_fn)(char*, const char*, >> apr_ssize_t, int, apr_size_t*); >> +#else >> +typedef apr_status_t (*encdec_fn)(char*, const char*, apr_ssize_t, int, >> apr_size_t*); >> +#endif >> > This uses assumptions about calling convention used on the Win32 platform. I > don't think it's a good thing. Do we really need an array of callbacks? May > just inline all these calls?
APR_DECLARE() seems to always __stdcall, as opposed to APR_DECLARE_NONSTD()'s __cdecl. But I barely understand these Windows subtleties, so yeah I can inline.. Regards; Yann.
