Cleanup: unused headers

Cleanup: add license header, use header guards

Doc: update to T73699 fix

It's important to refer to "other selected objects" in this case.

Doc: simplify T72883 fix

Code example made it seem setting the active object was more complex
than a single assignment.

RNA: Raise error assigning an active object which doesn't exist

Previously this silently cleared the view layers active object.

Fix T72883: Correct Context Usage

Fix T73699: Unhelpful error message when trying to knife project a manifold object

Revert "Sculpt: Remove partial viewport updates from sculpt stroke code"

This reverts commit 394b48029c5335381f774f83524ad6ee5cb986da.

Leaving this here does not affect performance in any significant way and it may be useful in the future.

Reviewed By: brecht

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

Fix T72690: Do not draw points behind the viewport camera in the paint cursor

In some situations the symmetry point may be behind the camera, so the
projection is inverted and it looks wrong. This avoids drawing points in
screen space when they are behind the camera.

Reviewed By: jbakker

Maniphest Tasks: T72690

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

Fix T66349: Add missing update for face orientation overlay

This forces the full geometry when that overlay is enabled.
I would rather not support this and leave that bug as a know issues/
limitation because this is terrible for performance and people may not
be aware that by enabling the option they are disabling the
optimization.

Reviewed By: jbakker

Maniphest Tasks: T66349

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

Cleanup: Fix warning (unused variable)

Cleanup: Rename `BKE_library_override_` functions to `BKE_lib_override_library_`

pqrt of T72604.

Updated install_deps to latest required OSL/LLVM.

CTest: Disable the cross test from the avxf unit tests.

Test fails on AVX2, I'll look into this but until the
cause is determined the test will be disabled

Refactor: move `Library`-specific functions into proper `BKE_library` file.

Even though we do not have much of those, this might change in the
future, and in any case having specific functions for this ID type in
generic `BKE_lib` area was really confusing.

Fix mismatch in BKE headers' multi-include safeguards naming.

Forgot to update those in recent `BKE_lib_` files renames.

Cleanup CLOG identifiers in BKE_lib' implementation files.

Merge branch 'blender-v2.82-release'

Fix unreported: Packing/unpacking errors don't mention tiled images

Thanks to @dfelinto for spotting this!

GPencil: Fix unreported missing annotation popover menu

This was introduced when annotations was splited.

Fix T73713: Tiled textures (UDIM) getting lost when saving the .blend but not the textures

The issue here is simple, IMA_SRC_TILED was missing from a check for whether the image should
be saved.

Note that tiled images won't be automatically saved if they have never been saved before.
For single images this is handled by packing them, but packing of tiled images isn't
supported yet.

However, in that case the file closing dialog will at least show a warning now instead of
completely ignoring tiled images.

Cleanup/Refactor: Move ID deletion into its own .c file.

Having functions defined in `BKE_lib_id.h` implemented into
`lib_remap.c` was confusing at best.

Besides trivial code splitting and header includes cleanup, had to add a
new `lib_intern.h` header for callbacks used by both remapping and
deletion code.

BLI: add utilities for defining non-movable and non-copyable classes

Structs and classes can subclass these member-free classes privately.
Then they become non-movable, non-copyable or both.

USD: Install USD library via install_deps.sh

This commit adds the download, extract, patch, build, and install of the
Universal Scene Description (USD) library to the `install_deps.sh`
script.

Reviewed By: mont29, LazyDodo

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

Cleanup: Add basic doc about each `BKE_main` and `BKE_lib` files.

Including expected prefixes for functions in those files.

Part of T72604.

Fix missed include update in recent BKE_library renaming.

BLI: improve various C++ data structures

The changes come from the `functions` branch, where I'm using
these structures a lot.

This also includes a new `BLI::Optional<T>` type, which is similar
to `std::Optional<T>` which can be used when Blender starts using
C++17.

Cycles: internal support for constant object/mesh attribute values

None are currently exported from Blender, this fixes the Cycles side
implementation.

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

UI: Graph Editor Show Cursor panel

Make the Graph Editor Cursor panel fit in and use the split layout.

Also removed the snap buttons, since they were only a subset of the snapping possibilities, all of which are in Key > Snap anyway, and also in the context menu.

Update the theme to use correct hierarchy progression like in other editors.

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

Reviewed by Brecht van Lommel

Cleanup/refactor: Rename `BKE_library` files to `BKE_lib`.

Note that `BKE_library.h`/`library.c` were renamed to
`BKE_lib_id.h`/`lib_id.c` to avoid having a too generic name here.

Part of T72604.

Tracking: Clarify calculation of camera matrix for object solver

It might be confusing why matrix is constructed from scratch rather
than using already calculated one.

Tracking: Cleanup, more clear variable name

Tracking: Fix usage PLACEHOLDER function

Don't use BKE_view_layer_context_active_PLACEHOLDER which is marked
as "never use this".

In fact, it isn't needed to lookup for camera in the tracking function
at all: camera object is always explicitly passed to it.

PY API doc generation: Add option for multi-tasking sphinx.

Fix T61138: Difficulty in aligning handles of Stroke of type Curve

The user has preference to select the center point for alignment.
But in this case, it was not available for selection.

Also prioritizes selection of the middle point over the handles.

Reviewed By: #user_interface, brecht

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

GPencil: Fix unreported missing annotation popover menu

This was introduced when annotations was splited.

Cycles: Fix linking error of new avxf unit tests

Was happening on macOS. The reason of this is because OpenImageIO
depends on boost, so it is to be passed to the linker after the
OpenImageIO libraries.

Fix: Missed this line in last commit

UI: Bone Custom Shape layout

The layout was somewhat poor - hard to see what relates to the custom object, and also weirdly we show the Wireframe toggle above the Custom Object control, even though it can only be active if a bone has a custom object set.

Instead, I grouped everything in a Custom Shape sub-panel and used greying out.

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

Reviewed by Brecht van Lommel

Fix T73681: Python exception adding Empty objects via `object_data_add`
with "Enter Edit Mode" enabled

It is valild to add Empties this way, but we cannot enter editmode on
those.

Fix T73696: Text editor scroll glitch

Scrolling a short text buffer could show the cursor offset.

Cleanup: spelling

Cleanup: sort file, struct lists

Cleanup: keymap formatting, unused arg

VSE: Add option to select handles with box selection

Patch adds an "Handle" option to the `SEQUENCER_OT_box_select` operator,
that allows to select the handles instead of whole strips.
Feature is mapped to Alt key modifier

A difference from the proposed design in T70730 is that covering the entire strip with the box actually selects both handles.

Reviewed By: iss

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

Merge branch 'blender-v2.82-release'

Fluid: Optimization for fluid domain obstacle boundary handling

This fix should especially improve liquid obstacles boundaries, i.e. help with the problem of particles sticking to the surface.

Fluid: Fixed file formats for script exporter

File formats in the exported scripts were using hardcoded file extensions and not the ones specified in the UI.

Fluid: Fixes for fluid guiding

Fluid guiding functionality was broken in the bake / read cache loop in fluid.c. Committing this to the release branch as otherwise fluid guiding would not have worked as expected (i.e. not at all).

Fluid: Updated manta pp files

Includes the OpenVDB read/write functions for int grids. This essential for the resume bake functionality in modular fluid caches.

Fix eigen/openmp link error in some build configurations

3D View: support axis views with axis-aligned roll

Previously any of the named views could not have any roll,
this commit supports roll as long as it's axis-aligned (90,180,270 deg).

This is useful for snapping to views,
an improvement on cebd025e02f11.

BLI_string_utils: add utility to join a string into a fixed buffer

3D View: check view quaternion alignment which orbiting

This makes auto-perspective work as expected
when orbiting out of a snapped view.

Cleanup/CMake: Fix superfluous warning of WITH_BOOST being turned off

The check for turning boost off did not actually check if WITH_BOOST
was on to start with leading to a superfluous warning during configuration.

Windows: Improve rebuild.cmd

Previously the MSVC environment was setup every time using vcvarsall.bat
when you ran you ran rebuild.cmd, Generally not an issue but after many
rebuilds on the same console, it grows the path environment variable
beyond what is supported and building breaks.

This patch adds a check to see if the environment is setup already
and skips the call to vcvarsall.bat

Also cleans up the double build in the msbuild's version of rebuild.cmd
install.vcxproj will build all that is needed, so no need to do a regular
build first.

Cycles: Add avxf unit tests.

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

Reviewers: brecht

Build_environment: Fix unwanted exports from oidn on windows.

oidn puts dllexport on all its functions causing the
blender binary to export these symbols.

this patch fixes this unwanted behaviour.

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

Reviewers: brecht , sergey

Fix Cycles error with hair and spatial splits after recent changes

UI: Cleanup Liquid Settings Code Style

This makes the code more consitent with the rest of blender

The `fractions_threshold` was changed to be active than enabled which is
also more consistent. This needs to be changed in other areas also.

Build_environment: Fix unwanted exports from libxml on windows.

libxml puts dllexport on all its functions causing the
blender binary to export these symbols.

this patch fixes this unwanted behaviour.

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

Reviewers: brecht , sergey

Build_environment: Freetype 2.10.1

Freeetype 2.9.1 tags dllexport on most of its functions so these
are now exported from the blender binary. (Same issue as D6563
which fixed it for USD)

Issue has already been fixed upstream so a simple version bump
fixes it.

This patch bumps freetype to 2.10.1

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

Reviewers: brecht , sergey

Fix Cycles embree render crash after recent refactor

Cleanup: fix compiler warning

Build_environment: OSL 1.10.9 / llvm 9.0.1

OSL 1.10.9 fixes osl-bug 866 [1] which is long standing issue
on windows where paths get un-escaped and osl breaks when you
install it to for instance c:\blender-tests\new-boolean

This patch bumps osl to 1.10.9, and since osl is llvm's
only consumer, llvm/clang were bumped 9.0.1

Removed some of the patches that were no longer needed

Builds and passes all tests on windows and linux

[1] https://github.com/imageworks/OpenShadingLanguage/issues/866

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

Reviewers: brecht

Merge branch 'blender-v2.82-release'

Update RNA Manual References

Cleanup: remove old VSE prefetching code.

Reviewed By: brecht

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

Merge branch 'blender-v2.82-release'

Fluid: Fixed cache reading issue with larger resolution noise grids

Important fix that needs to go into the release. The upscaled noise cache was not read into upscaled grids.

Code Quality Day: Comment formatting in bmesh_bevel.c

Fix T73660: Hide envelop option for Grease Pencil weights generation

As this feature is not supported yet, it's better to hide for grease pencil.
Differential Revision: https://developer.blender.org/D6777

remove poll for OBJECT_OT_parent_clear

This works on selected_editable_objects, so no active object mandatory.

After deletion of the active object and using slection tools such as
e.g. box select, the lack of an active object would prevent this
operator from being called without good reason.

Also cannot think of any other existing poll that would really make
sense (Editor type, ...).

ref T68975

Maniphest Tasks: T68975

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

Cleanup: add extern "C"

BLI: add index_range method to some data structures

This can be used to iterate over all indices with less code.

Cleanup: Improve usage of extern "C"

Cleanup: Animation, split driver evaluation into separate functions

No functional changes.

Cleanup: Animation, reduce indentation by reordering conditions

This turns error condition checks into precondition checks, grouping the
non-error functionality together towards the bottom of the function and
error-handling functionality towards the top.

No functional changes.

Fix T73625: GPencil array offset wrong whe use Scale or Rotation

This is not 100% a bug but a design change. The old method used the object origin as pivot point for Scale a nd Rotation, so when you moved the stroke in edit mode, the whole array ittems where offset because the pivot point distance changed.

Now, before applying scale and rotation, the stroke is moved to object origin to keep the offset when scale or rotate, so these transformations are done in stroke local space.

Cleanup: Rename `BKE_library_idmap` file to `BKE_main_idmap`

Part of T72604:
> Proposal: BKE_library and BKE_main API naming: prefixes conventions

Build optimization: Compile glew for size rather than speed with msvc.

Glew.obj is one of the more expensive targets we have to build clocking in
at 34.907 seconds.

The root cause [1] is the msvc inliner is not super thrilled with the
glewIsSupported supported function, this patch changes the buildflags
of extern_glew to optimize for size rather than speed.

Given glew is only in play at initalization time there will be no
performance impact.

Time to build glew.obj

Before: 34.907s
After :  1.961s

[1] https://developercommunity.visualstudio.com/content/problem/732941/slow-compilation-of-glewc-for-visual-studio-2019-x.html

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

Reviewers: brecht, sergey, fclem

Cleanup: `make format`

Remember to run `make format` after cleanups/renames/...

BuildSystem/Cleanup: Fix warning behaviour regarding library dependencies

Adding USD to a lite build fails to build due to boost errors, when you turn
boost on and rebuild still boost errors, boost was silently turned off since
it was not deemed needed. Once boost was forced on, it still fails due to TBB
being off.

This patch fixes:

- The Silent disabling of boost
- Add a check that USD is is not on before doing that
- move the TBB checks to a central location rather than the individual platform files
- Add USD to the TBB checks.

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

Reviewers: brecht, sybren

Merge branch 'blender-v2.82-release'

OpenVDB: Added missing macros in fluid wrapper file

These were needed to compile when WITH_FLUID=1 and WITH_OPENVDB=0.

Cleanup: use of 'unsigned'

- Replace 'unsigned' used on it's own with 'uint'.
- Replace 'unsigned const char' with 'const uchar'.

Cleanup: remove legacy OpenGL viewport clipping code

CodeCleanup: Remove unused studiolight algorithm

Code was originally added to test a different approach to calculate the
irradiance buffer. The approach was just to slow so we never used it.

This change will remove it from the code base

CodeCleanup: Workbench

Code was already hidden by a compile directive.

Cleanup: transform_convert comment and spacing

Cleanup: Animation, match parameter names in declaration with implementation

The implementation had more descriptive parameter names, so I copied those
to the declarations.

No functional changes.

Constraint: Fix forced request of evaluated camera

Constraint stack similarly to modifier stack is fully operate on what
have been given to it, without requesting original or evaluated IDs.
Validness of datablocks passed to constraint stack are to be handled
on dependency graph/evaluation stream levels.

Cleanup: Tracking, use LISTBASE_FOREACH

Makes loops declaration shorter and cleaner.

Cleanup: Tracking, reduce scope of variables

Cleanup: reduced indentations by returning/continuing early

No functional changes.

Cleanup: simplify Cycles primitive attribute map storage

Cleanup: split Cycles Hair and Mesh classes, with Geometry base class

Cleanup: remove unnecessary operations in Cycles curves export

Clearing and computing bounds is already done elsewhere.

Cleanup: split Cycles export into smaller files

Cleanup: export particle hair as a separate Cycles object