Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove all redundant header guards around #include directives #341

Closed
wants to merge 3 commits into from
Closed

Remove all redundant header guards around #include directives #341

wants to merge 3 commits into from

Commits on Feb 14, 2017

  1. Moved #includes in TGLContext.h inside headerguard

    @Teemperor
    Teemperor committed Feb 14, 2017
    Configuration menu
    Copy the full SHA
    60a6cf5 View commit details
    Browse the repository at this point in the history

  2. Remove all redundant header guards around include directives 

    Many headers contains redundant header guards around #include directives:

    #ifndef ROOT_TTree
    #include "TTree.h"
    #endif

This patch removes the #ifndef's around these includes as they
don't serve any pratical purpose anymore and are no longer part
of the current ROOT coding convention.

This patch also fixes the 153 typos that are contained in the
symbols of the #ifndef directives.
    @Teemperor
    Teemperor committed Feb 14, 2017
    Configuration menu
    Copy the full SHA
    f3a1600 View commit details
    Browse the repository at this point in the history

  3. Manually removed a few more remaining redundant header guards 

    The script didn't catch these in the previouis commit, so we
had to manually do this.
    @Teemperor
    Teemperor committed Feb 14, 2017
    Configuration menu
    Copy the full SHA
    307b4fb View commit details
    Browse the repository at this point in the history