[Libreoffice] [PATCH][PUSHED] Replace ENSURE_OR_* macros with regular code.
mmetz at adrian-broher.net
Tue Jan 24 01:43:28 PST 2012
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.
More information about the LibreOffice