just realized that i no longer need the additional redirection here:
[i'll send another patch including any other feedback i might get]
> @@ -159,6 +159,8 @@
> apr_pool_t *p,
> apr_uint32_t flags)
> {
> + struct apr_poll_pollset_t **pollset_p =
> + (struct apr_poll_pollset_t **)&pollset->p;
> if (flags & APR_POLLSET_THREADSAFE) {
> return APR_ENOTIMPL;
> }
> @@ -167,10 +169,10 @@
> return APR_ENOTIMPL;
> }
> #endif
> - pollset->p = apr_palloc(p, sizeof(apr_pollset_private_t));
> - pollset->p->pollset = apr_palloc(p, size * sizeof(struct pollfd));
> - pollset->p->query_set = apr_palloc(p, size * sizeof(apr_pollfd_t));
> - pollset->p->result_set = apr_palloc(p, size * sizeof(apr_pollfd_t));
> + *pollset_p = apr_palloc(p, sizeof(struct apr_poll_pollset_t));
> + (*pollset_p)->pollset = apr_palloc(p, size * sizeof(struct pollfd));
> + (*pollset_p)->query_set = apr_palloc(p, size * sizeof(apr_pollfd_t));
> + (*pollset_p)->result_set = apr_palloc(p, size * sizeof(apr_pollfd_t));
>
> return APR_SUCCESS;
> }
