On Tue, Mar 30, 2010 at 1:02 PM, Neil Conway <[email protected]> wrote: > Field seems unused to me -- +1 for removing it. Attached is a patch > against trunk that does this. > > Since this struct is exposed to client code, removing the field is > presumably only an option for APR2.
right, though it would be useful to declare in 1.x/apr_poll.h that the field is ignored for all apr 1.x; that would avoid any concern for devs that have to adjust their code for 2.x compatibility (i.e., delete lines that set that field) but don't want to break 1.x compatibility
