On Wed, Nov 11, 2009 at 17:11, Jeff Trawick <[email protected]> wrote: > At present, whatever was in errno at the time the dav_error {} was > created is treated as an apr_status_t by ap_log_rerror(). > > http://mail-archives.apache.org/mod_mbox/httpd-dev/200211.mbox/%[email protected]%3e > > dav_error {} should have an apr_status_t field instead of an errno > field; functions which create a dav_error should have an apr_status_t > parameter. > > If there's no direct apr_status_t representation of the error, the > caller will have to decide what to do (no magic portable solution > AFAIK, but no worse than today). > > Concerns?
dav_error was designed during the 1.3 series. APR wasn't even being discussed. So yeah... there is definitely a disconnect from "then" to "now". I'd be quite supportive of changing that, though (strictly speaking) that is probably an API change. Cheers, -g
