Hi, It seems that there is no objection. I'll do it in this week.
Thanks, -- kou In <20240828.100545.1738959907447472023....@clear-code.com> "[DISCUSS][C++] Indent #if (preprocessor directives)" on Wed, 28 Aug 2024 10:05:45 +0900 (JST), Sutou Kouhei <k...@clear-code.com> wrote: > Hi, > > How about indenting preprocessor directives for readability? > > Issue: https://github.com/apache/arrow/issues/43796 > PR: https://github.com/apache/arrow/pull/43798 > > For example: > > Before: > > ---- > #ifndef NDEBUG > #include "arrow/acero/options_internal.h" > #endif > ---- > > After: > > ---- > #ifndef NDEBUG > # include "arrow/acero/options_internal.h" > #endif > ---- > > > FYI: Google C++ style guide (our coding style is based on > it) doesn't require indent in preprocessor directives nor > deny it: > > https://google.github.io/styleguide/cppguide.html#Preprocessor_Directives > > ---- > // Good - directives at beginning of line > if (lopsided_score) { > #if DISASTER_PENDING // Correct -- Starts at beginning of line > DropEverything(); > # if NOTIFY // OK but not required -- Spaces after # > NotifyClient(); > # endif > #endif > BackToNormal(); > } > ---- > > > Thanks, > -- > kou
