libczi icon indicating copy to clipboard operation
libczi copied to clipboard

Published 20 hours ago • verified publisher icon ZEISS

Finalize CPPLINT

Open FelixS90 opened this issue 2 years ago • 0 comments

Is your feature request related to a problem? Please describe. With the completion of #32 we still have some cpplint issues left to fix (or permanently accept) that are currently filtered out in CPPLINT.cfg.

Describe the solution you'd like Apply some of the fixes primarily applied to Src/libCZI as part of #32 to other parts of the code like:

  • Src/CZICmd
  • Src/JxrDecode
  • Src/libCZI_UnitTests

Src/libCZI largely passed cpplint by applying the following filter: cpplint_screenshot

Any cpplint issues that are not going to be fixed (either because of feasibility or meaningfulness) should stay in the global CPPLINT.cfg. I strongly advise not to:

  • introduce nested config files or
  • exclude certain subdirectories from scanning

Additional context

image

FelixS90 avatar Feb 07 '23 10:02 FelixS90