[Bug 68849] Other: Add header guards to all include files

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Oct 21 17:32:12 CEST 2013


--- Comment #8 from Stephan Bergmann <sbergman at 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 ->
> 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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20131021/d4738d4f/attachment.html>

More information about the LibreOffice mailing list