Blender Git Commits

BLI: Switch threads implementation file to C++

Allows to use C++ primitives in the primitive implementation.

Clang-Tidy: Allow use with GCC but warn about possible failure

The compilation using GCC + Clang-Tidy succeeded for me and Sybren
(with some linker caveats for Sybren) but seems that it is doable
to make GCC + Clang-Tidy to officially work for Blender.

Now it should be possible to enable Clang-Tidy doing something like

make developer debug BUILD_CMAKE_ARGS='-DWITH_CLANG_TIDY=ON'

Clang-Tidy: Silence warning affecting newer code

For now Clang-Tidy should be passable with the codebase as is. This warning
will be addressed as quality day task.

Initial support of clang-tidy toolchain

Clang Tidy is a Clang based "linter" tool which goal is to help
fixing typical programming errors.

It is run as a separate compile step of every file, which slows
compilation down but allows to fully analyze the file the same
way as compiler does and catch non-trivial bugprone cases.

This change includes:

- CMake option called `WITH_CLANG_TIDY` which enables Clang Tidy
linter tool on all source in the `source/` directory.

This option is only available on Linux, as it is currently the
easiest platform to get the Clang Tidy toolchain to work.

- CMake module which is aimed to find latest available Clang Tidy.

- Set of rules which allows to have Blender fully compiled without
extra issues.

The goal of this change is to provide a base ground so that solving
all the warnings can happen later on, as a team effort.

It should be possible to use Clang Tidy side-by-side with both GCC
and Clang, but there seems to be some tweaks to be done in CMake to
make it really work for Blender. For now use Clang toolchain if
there are issues with GCC+Clang Tidy.

It will be worked on in the nearest future to bring seamless
experience for all configurations.

Currently there is no official way of getting Clang Tidy on macOS,
and on Windows there are some difficulties of hooking up Clang Tidy
from LLVM package to the MSVC compiler toolchain.

The actual warnings in the code will be addressed as a part of the
Code Quality Days, task T78535.

Differential Revision: https://developer.blender.org/D7937

Cleanup: spelling

UI: Add Edge Loopcut Slide to edge menu

This was exposed in the context menu but not the main edge menu.

Fix T78551 EEVEE: Fix memleak from renderpass ubo

Fix T78124 Overlay: Image: Camera background image transparency not working

This changes to premultiplied blending for all cases and put the premult
in the shader.

Fix T77559 Avoid wrong ob->imat leading to several bugs

This fixes T77559 Scale to zero in object mode unable to select

Fix T78527: GPencil Mirror modifier is inconsistent with Mesh Mirror (redo)

Simply the same code the regular mesh mirror modifier uses.

Differential Revision: https://developer.blender.org/D8188

UI: rename Alpha to Opacity

There was a discrepancy between the Tooltip of the Camera's Background Image that controls the opacity of the image and the operator's name that reads Alpha. The tooltip says **"Image opacity to blend the image against the background color"**, so it was renamed to Opacity to follow the tooltip.
The decision for this change is that the naming Alpha is not the most appropriated, since it has nothing to do if a loaded image has an embedded Alpha channel or not.

{F8540101}

Reviewed By: #user_interface, pablovazquez

Differential Revision: https://developer.blender.org/D7760

UI: Replace "Grease Pencil" with "Annotation" in user-facing options in the Clip editor

This revision affects the menu under: Clip > Track > Detect Features > Placement.
A bit of UI that was probably missed when the legacy GP was renamed to Annotation

{F8647693}

In this state, it may be confusing for the user.

Reviewed By: Blendify

Differential Revision: https://developer.blender.org/D8139

Cleanup: Remove useless duplicated lines

I spotted a duplicate struct declaration, so I had to check for other duplicated as well
There might be some other but i am not confident enough for deleting them

this regex search for duplicate ^(.*;)$n(1)$

Reviewed By: JacquesLucke

Differential Revision: https://developer.blender.org/D8146

UI: Dont abbriviate location & rotation

This resolves one of the last few areas where we still use inappropriate
abbreviations. Reading abbreviated words is usually slower, because
users must parse, guess and translate the words. Using abbreviations
such as 'rot' is also especially bad since it's a word in itself too.

The main advantage of abbreviations is that they are faster to *write*,
which just isn't a concern for text in the UI.

Differential Revision: https://developer.blender.org/D8174

GPencil: Cleanup - More rename from gp_ to gpencil_

Sculpt: Mask By Color

This tool generates masks based on the sculpt vertex colors by clicking
on the model, similar to automatic selection tools in image editing
software.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D8157

Cleanup: Fix build error with clang on windows.

Header and implementation signature for ED_object_add_duplicate
were not the same leading to a build error with clang on windows.

Fix T78525: Status bar not reset after Edit Voxel Size

Reviewed By: sergey

Maniphest Tasks: T78525

Differential Revision: https://developer.blender.org/D8181

UI: Use consistent order in ocean modifier

Ocean modifier: Expose eigenvectors for use as map

The eigenvectors in the ocean modifier (plus and minus) can be useful,
but are not exposed. Assuming the particle system was capable, the
eigenvectors could be used to drive spray emission velocities.

This exposes the controls to allow a map to be generated from these
eigenvectors. Currently, the values are mapped into a 0-255 range
similar to foam.

Differential Revision: https://developer.blender.org/D7182