Date: prev next · Thread: first prev next last
2010 Archives by date, by thread · List index


At 8:08am -0400 Fri, 22 Oct 2010, Thomas Klausner wrote:
On Thu, Oct 21, 2010 at 05:16:32PM +0200, Thomas Klausner wrote:
I'll switch diagnose.h (locally) to give " " for OSL_THIS_FUNC, as it
currently seems to do for everyone else.

To close this thread: I've pushed this change.

Have you enumerated everywhere this is used (in the code base). At the moment, I'm having a difficult time seeing the use (both logically and usefully) of a macro that defines a single space. At the point you're basically castrating that macro, why not just remove it entirely?

As they say, "code removed is code debugged," and we've already experienced some confusion over the handling of the use-related __PRETTY_FUNCTION__ non-macro.

Cheers,

Kevin

Context


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.