Blender Git Loki

UI Code Quality: Use derived struct for number buttons

For the man rationale behind this design, see 49f088e2d093. Further,
this removes users of uiBut.a1/uiBut.a2, which is a very ugly design
choice (hard to reason about).

Note that I had to do rather ugly, specific exceptions for the number
buttons in `ui_def_but_rna()`. But once all users of a1/a2 are removed,
this special handling shouldn't be needed anymore.
I also had to move a sanity check out of the button definition. It's now
moved into a new debug only sanity checking function executed when
finishing the layout definition (block end).

UI Code Quality: Store curve-button gradient type in derived struct

We want to move away from using `uiBut.a1`/`a2`, which is a hard to
reason about design.

Part of T74432.

Also correct comment.

Fix building on windows after cleanup changes in tiff.c

Cleanup: Remove redundant redeclaration in collada

No functional changes.

Cleanup: Clang-Tidy readability-inconsistent-declaration-parameter-name fix

No functional changes

Fix T80426: Crash when deleting custom orientation

Indices referencing other orientations were not being updated,

Fix build error on macOS after recent changes

Curve Profile: Miscellaneous cleanup

- Declare variables where they are initialized
- Use consistent variable and static function names
- Use helper functions more for common operations
- Remove use of BezTriple struct, use CurveProfilePoint instead
- Apply small simplifications to code in some situations

Cleanup: Clang-Tidy bugprone-copy-constructor-init fix

No expected functional changes.

Cleanup: fix compiler warnings for MSVC

CMake: refresh building and external library handling of Cycles standalone

* Support precompiled libraries on Linux
* Add license headers
* Refactoring to deduplicate code

Includes work by Ray Molenkamp and Grische for precompiled libraries.

Ref D8769

Cleanup: Use function pointers in sculpt gestures

This replaces the switch case and operation types with a
SculptGestureOperation struct with callbacks.

Reviewed By: sergey

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

Cleanup: Use function pointers in geometry extract

This replaces the switch case with function pointers to
extract masks or face sets.

Reviewed By: sergey

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

Cleanup: Clang-Tidy, describe current status of rules

No functional changes.

Cleanup: Clang-Tidy bugprone-incorrect-roundings fixes

Should cause no noticeable difference.

GPencil: Change Interpolate shortcut to Ctrl+E

Before the shortcut was Ctrl+Alt+E, but it's more logic remove the Alt.

Reviewed by: @mendio, @pepeland

GPencil: New option to set steps in Interpolate Sequence

Sometimes interpolate all frames is not convenient and it's better, for example, interpolate in twos.

The new parameter allows to define the number of frame for each step, by default is set to 1 as before.

{F8812621}

This is a request of animators to improve interpolate tools.

Reviewed By: mendio

Maniphest Tasks: T80190

Differential Revision: https://developer.blender.org/D8723
239b0b

Cleanup: more Clang-Tidy readability-const-return-type fixes

Building with `WITH_GMP=ON` revealed more const-return-type issues.

No functional changes.

Cleanup: typo fix in a comment

No functional changes.

Cleanup: Enable Clang-Tidy readability-qualified-auto rule

No functional changes. No changes except enabling the rule, even.