Re: [Libreoffice] [LibreOffice][Patch] Replace ASSERT and DBG_ASSERT with OSL_ENSURE + some translations, comment removal

Caolán McNamara <caolanm -AT- redhat.com>
Thu, 14 Oct 2010 09:11:26 +0100

On Wed, 2010-10-13 at 22:34 +0200, Jacek Wolszczak wrote:

http://wiki.services.openoffice.org/wiki/Debug_Levels. 

Some additional questions. What should I replace DBG_ERROR with?


Hmm, DBG_ERROR takes no condition, and just a message, I suggest that 
DBG_ERROR("horrible failure");
can become
OSL_ENSURE(false, "horrible failure");

Should RTL_LOGFILE_CONTEXT_AUTHOR as in http://bit.ly/buiBlA be updated
somehow as it contains a lot of hardcoded e-mail addresses?


*shrug*, nah, leave 'em alone I suggest.

C.

Context

[Libreoffice] [LibreOffice][Patch] Replace ASSERT and DBG_ASSERT with OSL_ENSURE + some translations, comment removal · Jacek Wolszczak
- Re: [Libreoffice] [LibreOffice][Patch] Replace ASSERT and DBG_ASSERT with OSL_ENSURE + some translations, comment removal · Cédric Bosdonnat
- Re: [Libreoffice] [LibreOffice][Patch] Replace ASSERT and DBG_ASSERT with OSL_ENSURE + some translations, comment removal · Caolán McNamara
  - Re: [Libreoffice] [LibreOffice][Patch] Replace ASSERT and DBG_ASSERT with OSL_ENSURE + some translations, comment removal · David Tardon

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.