How do you indent preprocessor statements?

Question

When there are many preprocessor statements and many #ifdef cascades, it\'s hard to get an overview since normally they are not indented. e.g.

#ifdef __WIN32__
#         


        

Answer 1

Just because preprocessing directives are "normally" not indented is not a good reason not to indent them:

#ifdef __WIN32__
    #include <pansen_win32>
#else
    #include <..>
    #ifdef SOMEOTHER
        stmts
    #endif
    maybe stmts
#endif

If you frequently have multiple levels of nesting of preprocessing directives, you should rework them to make them simpler.

Answer 2

First, make sure you actually need all the ifdef statements. Perhaps there's a way to refactor them to keep the number of nested if checks limited? Assuming you do need them:

It would be a good idea to separate the first portion (the includes) into a separate include file:

In your_header.h:

#ifdef __WIN32__
#include <pansen_win32>
#else
#include <..>
#endif

Then in the implementation file, you can then keep things more sane. Liberal blank lines and comments are the approach I've typically taken regarding non-indented #ifdef checks.

#ifndef __WIN32__
#ifdef SOMEOTHER

stmts

// SOMEOTHER
#endif

maybe stmts

// __WIN32__
#endif

All that said, there's no rule that you can't indent preprocessor checks if it makes the code more readable.

Answer 3

"All that said, there's no rule that you can't indent preprocessor checks if it makes the code more readable."

No, but in a world where style is controlled and the code is adjusted to match it would be nice to enable the style to indent #if statements like if statements.

While #if might technically be a different language it is part of the C/C++ specification and as it is not a letter for letter match of any current keywords there is no reason that #ifs can not be accommodated for in the styling.

Answer 4

This question already has lots of valid answers. For people that want a visual image here it is.

In visual studio go to options search for indent and select your language. In my case it is c++. As you toggle between the options visual studio will show you an example bellow.

Answer 5

As you, I didn't make my mind yet about the best way to indent, but I found in more than one place this alternative indentation in which the # is placed always at the first column and just the keyword is indented:

#ifdef __WIN32__
#  include <pansen_win32>
#else
#  include <..>
#endif

In Visual Studio, when you type # as the first character it always brings its indentation to the left, so it seems that MS either favors never indenting preprocessor statements, or using the above format.

The big problem is when you have non-preprocessor and preprocessor statements mixed and indentation applied. It's hard to make code that looks good, no matter which option:

option (a)

for (...)
{
  if (foo)
  {
    if (bar)
    {
#ifdef __WIN32__
      c = GetTickCount();
#else
      c = clock();
#endif
    }
  }
}

option (b)

for (...)
{
  if (foo)
  {
    if (bar)
    {
      #ifdef __WIN32__
      c = GetTickCount();
      #else
      c = clock();
      #endif
    }
  }
}

option (c)

for (...)
{
  if (foo)
  {
    if (bar)
    {
#     ifdef __WIN32__
        c = GetTickCount();
#     else
        c = clock();
#     endif
    }
  }
}

At this point, it becomes a question of personal taste, as so many other indentation styles.