Dženan Zukić

Results 653 comments of Dženan Zukić

I am running the build script with doxygen updated to 1.9.5, so we don't have to wait until tomorrow to see the updated output.

The second suggestion would require us to move to a newer `clang-format` (14 or later). We are currently at 8.0.0. Moving to a newer one shouldn't be a big change....

Doxygen 1.9.5 produces 100+ new warnings of style: Generating doc[Documentation/Doxygen/NeighborhoodIterators.dox:96](https://github.com/InsightSoftwareConsortium/ITK/blob/master/Documentation/Doxygen/NeighborhoodIterators.dox#L96): warning: Unexpected token TK_HTMLTAG found as part of a title section. All warnings [here](https://open.cdash.org/viewBuildError.php?type=1&onlydeltap&buildid=8345942). @jhlegarreta should we keep this version?

Adding `end of class` comment does trigger the described workaround: ```cpp /** Sets the value of the image buffer at the current index value to the * specified value. */...

I still think that moving to latest stable `clang-format` (currently [15.0.7](https://github.com/llvm/llvm-project/releases)) is the best solution.

The current version ignores (does not support) the config setting we need. As per this [SO answer](https://stackoverflow.com/a/70732858/276168), we need `SeparateDefinitionBlocks`, which was introduced with clang 14. So we need version...

This does reduce number of `warning: unbalanced grouping commands` from 92 down to 20.

I updated Doxygen on blaster to latest (1.13.2). Merging this PR sounds like a low-risk step in the right direction. Let's give people a few days to review (or maybe...