[Bf-committers] Code Quality Day - 4/Dec

[Ray Molenkamp]

For the windows folks,

I have enabled clang-tidy for windows so you can now join in
on the fun, but there are some caveats that are important to
be aware of.

1- It will only be configured if WITH_CLANG_TIDY is on in cmake.

2- Only supported in the IDE, no ninja or command line support at
   this moment in time.
 
3- Disabled on build by default, it is really slow running the
   compiler twice, and it emits still a fair number of errors on
   bad code inside windows specific ifdefs. For those reasons
   you have to manually run it from the analyze sub menu.
 
If you run into issues using it feel free to prod me on chat

have fun tomorrow!

--Ray
On 2020-12-03 8:38 a.m., Sergey Sharybin via Bf-committers wrote:
> Hi,
>
> This Friday we have December's code quality day [1].
>
> The full list of approved projects for the quality day is combined in a
> dedicated task [2].
>
> Last day we finished what one might consider essential Clang-Tidy
> integration warnings. There are now modernize category warnings enabled,
> which helps us bring code to 2020 standards [3].
>
> From the last day there is still [4]. It was partially tackled, but there
> are still some ideas which can end up in a nice quality improvement.
>
> There are again some ideas I have in mind of improving the build process.
> Let's talk about it tomorrow!
>
> I would also encourage every module to come with ideas (and solutions ;) to
> what they consider a quality/technical-depth improvement!
>
> Everyone is welcome to help!
>
> For feedback on what to do, code review, and everything else we should use
> #blender-coders in blender.chat.
>
> [1] - https://wiki.blender.org/wiki/Style_Guide/Code_Quality_Day
> [2] - https://developer.blender.org/T73586
> [3] - https://developer.blender.org/T78535
> [4] - https://developer.blender.org/P1756
>
> Have fun tomorrow,
> -Sergey-
> --------------------------------------------------------------------
> Sergey Sharybin - sergey at blender.org - www.blender.org
> Principal Software Engineer, Blender
> Buikslotermeerplein 161, 1025 ET Amsterdam, the Netherlands
> _______________________________________________
> Bf-committers mailing list
> Bf-committers at blender.org
> https://lists.blender.org/mailman/listinfo/bf-committers