@amadio commented on this pull request.
> +//! Return advanced error information about the last error encountered by +//! this object. Only invoked if the XrdOSS indicates it has feature XRDOSS_AERR +//! The advanced error feature provides the ability for more error information +//! than is possible from the typical error code (e.g., unstructured error +//! messages). +//! +//! @return Error object associated with the last error. Caller does not own +//! the returned memory. Lifetime of the returned object is only guaranteed +//! until the next method call. May return nullptr even if an error occurred. +//----------------------------------------------------------------------------- + +#if XRDOSS_ERR +virtual const XrdOucErrInfo *getError() const; +#else +const XrdOucErrInfo *getError() const; +#endif
I am not a fan of changing interface in this manner. I think it's better to have a clean patch and add it only in the next major version.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1