[Libreoffice] [PATCH][PUSHED] Replace ENSURE_OR_* macros with regular code.

Marcel Metz mmetz at adrian-broher.net
Tue Jan 24 01:43:28 PST 2012


Hello Stephan,

On 24.01.2012 09:30, Stephan Bergmann wrote:
> However, I have another question:  I notice that, while most
> occurrences  are replaced with SAL_WARN, some were left as OSL_ENSURE
> (which I changed to SAL_WARN; one missed a comma, anyway) and some
> are changed to SAL_INFO.  Was there some rationale for that?

No, unfortunately none aside from my sloppiness. Even after reviewing it
twice it does seem like I don't get it right. :/
Thanks for your review Stephan.

regards Marcel


More information about the LibreOffice mailing list