60

I am currently working on a big project and maintaining all those include guards makes me crazy! Writing it by hand is frustrating waste of time. Although many editors can generate include guards this doesn't help much:

  1. Editor generates guard symbol based on a filename. The problem occurs when you have headers with the same filename in different directories. Both of them will get the same include guard. Including directory structure into the guard symbol would require some fancy approach from the editor, since slashes and backslashes in the macro are not the best thing.

  2. When I have to rename a file I should rename all the include guards as well (in the ifndef, define and ideally endif's comment). Annoying.

  3. Preprocessor is flooded with tons of symbols without a clue what they mean.

  4. Nevertheless definition is included once, compiler still opens header every time it meets header inclusion.

  5. Include guards don't fit into namespaces nor templates. In fact they are subverting namespaces!

  6. You have a chance that your guard symbol won't be unique.

Maybe they were acceptable solution in times when programs contained less than 1000 headers in single directory. But nowadays? It is ancient, it has nothing to do with modern coding habits. What bothers me the most is that this issues could be almost compeletly solved by #pragma once directive. Why is it not a standard?

9
  • 41
    I fear for any project that has more than 1000 headers in a single directory. Nov 8, 2009 at 9:00
  • 6
    include guards aren't a big deal. just make them random long strings and unless your source of randomness is abysmal you won't run into any problems.
    – Yuliy
    Nov 8, 2009 at 9:05
  • 9
    Then why you have directories at all? Just give every file different name and directories are useless ;) Isn't this a proove that c++ inclusion system is ancient? It can not utilize directories in efficient way. Giving the same name for files in different locations is natural and a common practice in other languages. In Java for example you have plenty of classes with the same names, thus in the same file names since Java uses one class per file convention, but in different namespaces.
    – mip
    Nov 8, 2009 at 21:15
  • 5
    No it is not. If you have for example a plugin system it is very likely that some filenames will be repeated. What's confusing? You have directory and you know on which part of application you are working.
    – mip
    Nov 9, 2009 at 1:12
  • 2
    While they are at it, why don't the ISO standards include a pony? And I was promised flying cars!
    – asveikau
    Nov 14, 2009 at 4:40

9 Answers 9

55

A directive like #pragma once is not trivial to define in a fully portable way that has clear an unambiguous benefits. Some of the concepts for which it raises questions are not well defined on all systems that support C, and defining it in a simple way might provide no benefit over conventional include guards.

When the compile encounters #pragma once, how should it identify this file so that it doesn't include its contents again?

The obvious answer is the unique location of the file on the system. This is fine if the system has unique locations for all files but many systems provide links (symlinks and hardlinks) that mean that a 'file' doesn't have a unique location. Should the file be re-included just because it was found via a different name? Probably not.

But now there is a problem, how is it possible to define the behaviour of #pragma once in a way that has an exact meaning on all platforms - even those that don't even have directories, let alone symlinks - and still get the desirable behaviour on systems that do have them?

You could say that a files identity is determined by its contents, so if an included file has a #pragma once and a file is included that has exactly the same contents, then the second and subsequent #includes shall have no effect.

This is easy to define and has well defined semantics. It also has good properties such that if a project is moved from a system that supports and uses filesystem links to one that doesn't, it still behaves the same.

On the downside, every time an include file is encountered containing a #pragma once its contents must be checked against every other file using #pragma once that has already been included so far. This implies a performance hit similar to using #include guards in any case and adds a not insignificant burden to compiler writers. Obviously, the results of this could be cached, but the same is true for conventional include guards.

Conventional include guards force the programmer to choose a macro that is the unique identifier for an include file, but at least the behaviour is well-defined and simple to implement.

Given the potential pitfalls and costs, and the fact the conventional include guards do work, it is not surprising to me that the standards committee didn't feel the need to standardize #pragma once.

21
  • 9
    Furthermore, if I remember correctly, #pragma is a preprocessor directive to allow for vendor-specific preprocessor commands, so it's kind of an oxymoron to standardize a pragma.
    – blwy10
    Nov 8, 2009 at 13:00
  • 8
    "The obvious answer is the unique location of the file on the system. This is fine if the system has unique locations for all files but many systems provide links (symlinks and hardlinks) that mean that a 'file' doesn't have a unique location. Should the file be re-included just because it was found via a different name? Probably not.".................................................................. Just follow links, they point to explicit file name! Really, this shouldn't be a tremendous task for the compiler building team.
    – mip
    Nov 8, 2009 at 21:32
  • 2
    On a typical Unix filesystem, you can tell from a filename whether there are other hardlinks that link to a given file, there is usually no reverse mapping to tell you what other filenames also link to this file, indeed a link might be just an open file handle with no name. Even if there is another name that links to a file there is no 'primary' name, so either name is just as good... unless someone unlinks one of the names because it was only a temporary filename.
    – CB Bailey
    Nov 8, 2009 at 22:14
  • 26
    I do not buy it. It is not a problem to implement on each system in a logical way (e.g. linux has inodes, on windows you can get similar information for a file), and from the point of view of the standard it is not a problem to standardize it with some extremely vague implementation-defined way, just as the meaning of #include is also implementation defined. Nov 17, 2012 at 18:16
  • 3
    @ybungalobill: According to the comments, such renumbering is seen when the OS has to go over the network to reach headers. (I suspect it also depends on the filesystem) Deletion/replacement is not part of this case. Apr 9, 2015 at 16:32
19

Include guards are definitely an annoyance, and C should have originally been designed such that headers would be included once by default - requiring some special option to include a header multiple times.

However, it wasn't ,and you're mostly stuck with having to use include guards. That said, #pragma once is pretty widely supported so you might be able to get away with using it.

Personally, I solve your 1st problem (similarly named include files) by adding a GUID to the include guard. It's ugly, and most people hate it (so I'm often forced to not use it at work), but your experience shows that the idea has some value - even if it is hideously ugly (but then again the whole include guard thing is kind of a hack - why not go whole hog?):

#ifndef C_ASSERT_H_3803b949_b422_4377_8713_ce606f29d546
#define C_ASSERT_H_3803b949_b422_4377_8713_ce606f29d546

// blah blah blah...

#endif

I've heard that compilers don't actually reopen header files that have include guards (they've learned to recognize the idiom). I'm not sure if that's true (or to what extent it's true); I've never measured it. I also don't worry about it, but my projects aren't so huge that it's a problem.

My GUID hack pretty much solves items 1, 5, and 6. I just live with items 2, 3, and 4. Actually, for item 2 you could live without renaming the include guard macro when you rename the file as the GUID will ensure it remains unique. In fact, there's no reason to incorporate the filename at all with the GUID. But I do - tradition, I suppose.

4
  • 1
    I agree that this is the best solution, unfortunately I'm using Eclipse which is a great tool but has terrible suport for include guards :/
    – mip
    Nov 8, 2009 at 10:02
  • 1
    I don't agree that this solved namespaces issue. As one may guess namespaces were invented to avoid names collision, and set natural names free again. This is achieved by wrapping all potentialy colliding code into namespace brackets. And of course include guards are exceptional to this rule. So it ends up in some kind of missconception. While structures are going to have more human names, include guards are going into exactly opposite direction. Resolving their name conflicts is left to IDE/scripts/whatever. This isn't pretty philosophy in my opinion.
    – mip
    Nov 8, 2009 at 10:09
  • 3
    I agree that it's not pretty, but it it wasn't meant to be an ideal solution, but a pragmatic one. It does pollute the global namespace, but does so with names that are designed to be universally unique, and the names are used in exactly one place (on 2 lines, but right next to each other). Actually, it's overkill for most people (most projects get by with much simpler guard names). Personally, I see little downside to using them, other than the ugliness - but I don't have to interact with them much, so it's not a bad tradeoff. It would have been better if C didn't need guards, but it does. Nov 8, 2009 at 17:53
  • 4
    @MichaelBurr: I would say #pragma once is the "pragmatic" solution. Jun 4, 2012 at 9:31
14
  1. How often do you have to add an include file to this project? Is it really so hard to add DIRNAME_FILENAME to the guard? And there is always GUIDs.
  2. Do you really rename files that often? Ever? Also, putting the GUARD in the #endif is just as annoying as any other useless comment.
  3. I doubt your 1000 header files guard defines are even a small percentage of the number of defines generated by your system libraries (especially on Windows).
  4. I think MSC 10 for DOS (20+ years ago) kept track of what headers were included and if they contained guards would skip them if included again. This is old tech.
  5. namespaces and templates should not span headers. Dear me, don't tell me you do this:

    template <typename foo>
    class bar {
    #include "bar_impl.h"
    };
    
  6. You said that already.

8
  • 4
    If the headers are as badly laced with conditionals as the ones I work with, having a comment after #endif can be valuable for seeing more easily which condition you are seeing the end of (especially when the start of the conditional is several hundred lines up the file). Nov 9, 2009 at 2:41
  • 1
    I can think of use cases like those above (look at boost source code to see namespaced includes in action) but then there won't be any include guards at play, probably.
    – MP24
    Nov 9, 2009 at 9:54
  • If it's that bad a problem, move the #define to just before the #endif. Then you don't need the comment. Nov 9, 2009 at 13:59
  • 1
    Another way to avoid having to edit the comment at the guard's #endif when you change the macro name is to use a generic comment: #endif /* close include guard */ It's not going to be very confusing, since there's only ever one include guard section and the endif for an include guard is pretty much always the last line in the header anyway (so a comment is largely redundant anyway, but I can understand wanting one). Nov 10, 2009 at 1:23
  • 2
    @JAB, my template example is not "silly" it's frightening, which is why I included it as an example of something NOT to do. May 18, 2014 at 14:40
10

As was already noted, C++ Standard should account for different development platforms some of which may have limitations making #pragma once support impossible to implement.

On the other hand, support for threads was not added for the similar reason previously, but newer C++ Standard includes threads nevertheless. And in the latter case we may cross-compile for a very limited platform, but development is done on a full-fledged platform. Since GCC supports this extension, I think, the real answer to your question is that there is no interested party in pushing this feature into C++ Standard.

From practical standpoint, include guards caused our team more trouble than the incompliance of #pragma once directive. For instance, GUID in include guards do not help in case if file is duplicated and later both copies are included. When using only #pragma once we get duplicate definition error and can spend time unifying source code. But in case of include guards the problem may require run-time testing to catch, e.g. this happens if copies differ in default arguments for function parameters.

I avoid using include guards. If I have to port my code to a compiler without #pragma once support, I will write a script which will add include guards to all header files.

4

The problem occurs when you have headers with the same filename in different directories.

So you have two headers which are both called ice_cream_maker.h in your project, both of which have a class called ice_cream_maker defined in them which performs the same function? Or are you calling every class in your system foo?

Nevertheless definition is included once, compiler still opens header every time it meets header inclusion.

Edit the code so you don't include headers multiple times.

For dependent headers (rather than the main header for a library), I often use header guards which are like this:

#ifdef FOO_BAR_BAZ_H
#error foo_bar_baz.h multiply included
#else
#define FOO_BAR_BAZ_H

// header body

#endif
1
  • Yeah I know, preprocessor is powerful. However this method brings even more passive code. Why not just #pragma once and you have all this in a single line, which may be blindly pasted even by very primitive editor. In modularity aspects C++ is in early middle ages. Nor Java, nor PHP, nor Perl, nor Python, nor Smalltalk, nor Rebol, nor probably any modern language do inclusions in the way C/C++ does. Even Turbo Pascal got nicer system of units.
    – mip
    Nov 8, 2009 at 10:36
3

IIRC, #pragma anything is not part of the language. And it shows up in practice, a lot.

(edit:completely agree that the inclusion and linking system should have been more of a focus for the new standard as it is one of the most obvious weaknesses, in 'le this day and age' )

5
  • 6
    #pragma STDC FP_CONTRACT ON ??
    – CB Bailey
    Nov 8, 2009 at 10:37
  • Don't know Charles, as I can't look it up easily right now.. I'd say it is a compiler directive, rather than language thing but could be way wrong here.. Nov 8, 2009 at 10:45
  • 2
    I was giving an example of a #pragma something that is defined in the language. ISO/IEC 9899:1999 6.10.6/2 is where the standard C #pragmas are listed (if you wanted to look them up).
    – CB Bailey
    Nov 8, 2009 at 11:38
  • of course you could be right.. perhaps others can chime in as well. anyway, it seems pretty chaotic with: "1996 Any such pragma that is not recognized by the implementation is ignored " "1995 The behavior might cause translation to fail or cause the translator or the resulting program to behave in a non-conforming manner. " "2001 an implementation is permitted to behave as if it were the standard pragma, but is not required to." Nov 8, 2009 at 11:50
  • perhaps those are special, perhaps not. But reading the above just doesn't give me confidence to say it is a not compiler/implementation-dependent interpretation, thus possibly disqualifying it from being defined in the language. all of theoretical value only of course, as the practice has obviously taken on the opposite direction with each vendor shooting ever-increasing pragma-related warnings at each other. Nov 8, 2009 at 12:02
2

You can probably avoid name collisions without resorting to random strings by setting the include guard to contain the name of the class and namespace that's in the file.

Besides, #pragma once is supported by both MS compilers and GCC for quite some time now so why does it bother you that it's not on the ISO standard?

2
  • 2
    I for one am actively using a compiler that doesn't support #pragma once. The OP may need to support such things. Nov 8, 2009 at 9:28
  • 1
    Because it is not an ISO standard and so far it is not portable. Moreover GCC had been marking it as obsolete feature for quite a long time. They still don't recommend to use it...
    – mip
    Nov 8, 2009 at 9:31
2

A pragmatic solution:

1) choose some consistent guard-naming policy (e.g. path relative to project root + file name, or whatever else you choose). Include possible exceptions for third-party code.

2) write a program (a simple python script would do) to walk recursively the source tree and verify that the guards all conformal to the policy. And whenever the guards are wrong, output a diff (or sed script, or whatever else) that the user can easily apply to fix. Or just ask for a confirmation and make changes from the same program.

3) make everyone on the project use it (say, before submitting to source control).

1

I think right way to allow to do multiple include with special pragma only and disallow to multiple include by default for example:

#pragma allow_multiple_include_this_file

So since you asked why. Did you send your proposal to standart developers? :) I'm not send too. Can one be a reason?

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.