Skip to content

linter complains about indented macros #204

@JeffGarland

Description

@JeffGarland

I am attempting to have structure in a macro

  // if the config.hpp file does not exist because this is in godbolt
  // or another context where cmake is not run, default to the correct
  // answer.
  #if (_MSC_VER)
    #define BEMAN_ANY_VIEW_NO_UNIQUE_ADDRESS() [[msvc::no_unique_address]]
  #else 
    #define BEMAN_ANY_VIEW_NO_UNIQUE_ADDRESS() [[no_unique_address]]
  #endif

Expected Behavior

Linter should ignore -- I thought this was clang-format, but turning that off had no impact. So it's whatever python the review-dog is running.

Current Behavior

The linter wants to flatten the structure to no spaces --

@@ -6,14 +6,14 @@
#if !defined(__has_include) || __has_include(<beman/any_view/config.hpp>)
#include <beman/any_view/config.hpp>
#else

  • // if the config.hpp file does not exist because this is in godbolt
  • // or another context where cmake is not run, default to the correct
  • // answer.
  • #if (_MSC_VER)
  • #define BEMAN_ANY_VIEW_NO_UNIQUE_ADDRESS() [[msvc::no_unique_address]]
  • #else
  • #define BEMAN_ANY_VIEW_NO_UNIQUE_ADDRESS() [[no_unique_address]]
  • #endif
    +// if the config.hpp file does not exist because this is in godbolt
    +// or another context where cmake is not run, default to the correct
    +// answer.
    +#if (_MSC_VER)
    +#define BEMAN_ANY_VIEW_NO_UNIQUE_ADDRESS() [[msvc::no_unique_address]]
    +#else
    +#define BEMAN_ANY_VIEW_NO_UNIQUE_ADDRESS() [[no_unique_address]]
    +#endif
    #endif

Additional Discussions

Any insight into what tool to actually fix would be appreciated.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions