openfhe-development icon indicating copy to clipboard operation
openfhe-development copied to clipboard

Published 20 hours ago • verified publisher icon openfheorg

Styling errors with cpplint

Open sararv22 opened this issue 2 years ago • 1 comments

cpplint fails while running pre-commit after clang-format passes for styling issues such as "Should have a space between // and comment" errors. This could possibly be fixed using SpacesBeforeTrailingComments from https://clang.llvm.org/docs/ClangFormatStyleOptions.html automatically with clang-format instead of fixing them manually.

sararv22 avatar Aug 05 '22 08:08 sararv22

SpacesBeforeTrailingComments is not the correct option for this case. The correct option is SpacesInLineCommentPrefix. it is available for clang-13 or later versions.

dsuponitskiy avatar Aug 15 '22 18:08 dsuponitskiy