https://bugs.freedesktop.org/show_bug.cgi?id=68849
--- Comment #8 from Stephan Bergmann <sbergman@redhat.com> ---
(In reply to comment #7)
I'll use the following scheme for all new or changed header guards:
All uppercase: INCLUDED_%modulename_%filename_%extension
Examples:
basic/source/runtime/rtlproto.hxx -> INCLUDED_BASIC_RTLPROTO_HXX
embeddedobj/source/inc/closepreventer.hxx ->
INCLUDED_EMBEDDEDOBJ_CLOSEPREVENTER_HXX
sd/source/ui/inc/dlgass.hxx -> INCLUDED_SD_DLGASS_HXX
With this INCLUDED prefix we will always be able to identify this as include
guard. This will be an advantage for future scripting efforts.
FYI, the convention that I use is to use INCLUDED_FOO_BAR_HXX for a global
header include/foo/bar.hxx under include/ and the longer
INCLUDED_FOO_SOURCE_BAR_BAZ_HXX for anything else like foo/source/bar/baz.hxx,
in an attempt to minimize the chance of clashes.
--
You are receiving this mail because:
You are on the CC list for the bug.
Context
- [Bug 68849] Other: Add header guards to all include files · bugzilla-daemon
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.