[Bug 82577] get rid of prex.h / postx.h wrapper headers
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Sun Aug 17 00:15:18 PDT 2014
https://bugs.freedesktop.org/show_bug.cgi?id=82577
--- Comment #3 from Tor Lillqvist <tml at iki.fi> ---
One thing that we need consensus on is whether to use a "real" C++ namespace or
a "C-style" one, i.e. just prefixing the conlicting LibreOffice type names with
some short string, like "Vcl".
I guess the C++ namespace would be better from a C++ orthodoxy point of view,
but what should the namespace be? "vcl::" ? "org::libreoffice::vcl::" (brrr)?
In any case, we *don't* want to repeat the current disaster of inconsistent
"using" declaration, varying from one source file to another. Would using a C++
namespace have the benefit that it would be enough to just add a "using
namespace vcl" (or whatever) in some header, and only those few source files
that actually refer to the identically-named X11 types would need to add a ::
prefix to those?
Personally I wouldn't mind using a "C-style" prefix, but then I am well known
to not really be that huge a C++ fan.
For the cases where there are clashes with *macros* (I guess mostly for
Win32?), the "C-style" identifier prefix (or even renaming our identifier
completely) is the only solution, right?
--
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20140817/1ca56208/attachment.html>
More information about the LibreOffice
mailing list