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


On 19.03.2013 00:02, Norbert Thiebaud wrote:
On Mon, Mar 18, 2013 at 11:17 AM, Lubos Lunak <l.lunak@suse.cz> wrote:
On Monday 18 of March 2013, Tor Lillqvist wrote:
Sounds great to me too; though of course I'd prefer to keep the diff
smaller and not replace all the header guards:

#ifndef INCLUDED_OOXML_FAST_CONTEXT_HANDLER_HXX
#define INCLUDED_OOXML_FAST_CONTEXT_HANDLER_HXX
...
Oh the other hand, it would be nice if the header guards were
consistently named... and didn't start with an underscore (such
identifiers are supposed to be reserved for the language and/or OS
implementation, unless I am mistaken). Somebody just needs to come up
with a consistent naming style and do it (with some nice script). (The
style of above example would be fine with me.)

  Yes, a script should be enough for this. And yes, "somebody" just needs to do
it :).

otoh, #pragma once is supported by all the compiler we use isn't it ?
so if we are going to change it, why not use that instead.

There are some discussions about that on the Internet. Most interesting: Some kind of benchmark comparison at

http://tinodidriksen.com/2011/08/31/cpp-include-speed/

Looks like header guards as we have them are the best solution on gcc, but the worst for MSVC and no combination would be acceptable compared to 'plain' header guard with gcc.

Thomas

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.