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.