Cleanup: use BLI_compiler_compat.h for BLI_INLINE

Cleanup: renmae ePaintTexture(Projective) -> 3D

Matches ePaintTexture2D, less verbose.

Fix T56055: color discrepancy between viewport and render for Filmic transforms.

Fix T57388: Blender Internal + Freestyle viewport render wrongly using FSAA.

Fix assert rendering hair tests on some systems.

Fix assert rendering with denoising, after recent changes.

Fix ghash masking out upper bits on 64bit systems

The code this was taken from assumes a 'size_t' result,
which isn't the case here.

In practice the bucket distribution wasn't bad,
even so this was a nop so best fix.

Fix T57529: 2D image paint fill tool not taking into account alpha.

Cleanup: move progress utility module into bpy_extras

Try avoid having too many toplevel modules with generic names.

Cycles: Fix wrong BVH used when disabling AVX2 in debug settings

Mainly useful for debugging. Previously, when AVX2 was disabled
in the debug panel but BVH layout was kept on BVH8 nothing was
rendered.

Needed to make it so supported BVH layout mask for devices is
queried in "dynamic", so it is possible to use DebugFlags there.

make.bat: fix unquoted variables.

causing build issues for some users.

Fix build error on Windows 32bit, alignment was wrong.

UI: add uiItemMenuFN which frees it's argument

Cycles: Support generating Denoising passes without actually denoising

Needed for the animation denoiser since the denoising filter is done separately there.

Reviewers: brecht, sergey

Reviewed By: brecht

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

Cycles: more detailed tooltips for cryptomatte options.

Fix assert weight painting after undo

Modifier: mask threshold option

D3834 by @Allosteric

Fix snaps appearing in system bookmarks on Linux.

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

Fix Linux build after Cryptomatte commit.

Cycles: Added Cryptomatte output.

This allows for extra output passes that encode automatic object and material masks
for the entire scene. It is an implementation of the Cryptomatte standard as
introduced by Psyop. A good future extension would be to add a manifest to the
export and to do plenty of testing to ensure that it is fully compatible with other
renderers and compositing programs that use Cryptomatte.

Internally, it adds the ability for Cycles to have several passes of the same type
that are distinguished by their name.

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

Fix T57393: Cycles OSL bevel and AO not working after OSL upgrade.

Cycles: Expose noisy image pass by default when rendering with denoiser

Apparently quite a few users would like to have the noisy pass available when using the denoiser, and since it's being generated anyways we might as well expose it by default.

Reviewers: brecht

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

Cycles: Overhaul ensure_valid_reflection to fix issues with normal- and bumpmapping

This function is supposed to prevent the black artifacts caused by strong normal- or bumpmapping, but failed in some cases.

Now the code correctly handles all test files and previous issues I am aware of and also has extensive comments describing
the algorithm and the math behind it.

Basically, the main problem was that there can be multiple valid solutions that fulfil the reflection angle criterium,
but I had assumed that only one would exist and therefore simply picked the first solution with a positive term in srqt().
Now, the code uses additional validity checks and a simple heuristic to pick the best valid solution.

Additionally, the code messed up very shallow reflections even if the normal map strength was zero due to the constant
limit for the outgoing ray angle, which caused shallow incoming rays to fail the initial test even when reflected directly
on Ng. Now, the code accounts for this by reducing the threshold in the case of a shallow incoming ray, ensuring that at
least N=Ng is always a valid solution.

Reviewers: brecht

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

WM: minor changes to cursor API

Cleanup: unused vars, correct bad (unused) logic

Fix (unreported) bad halding of ID usercounts when fully copying a scene.

I18n: tweak/fix exception catching code of pgettex() wrapper.

Related to T57066, rather unlikely to fix core of the issue, but may
solve crash itself.

Same as rBf22385f28ec8 (did it by mistake in 2.8 first :/ ).

BMesh: inline index lookups

For release builds this is now the same as indexing the array directly.

Windows: Enable python debugging in Visual Studio.

see D3817 for technical details, and https://wiki.blender.org/wiki/Tools/Debugging/Python_Visual_Studio for a end user quick-start guide.

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

Tentative fix for T57066: Daily build of Blend 2.79 crashes on OSX Mojave.

Looks like we need to explicitly set i18n language to default value on
some OSs... Unless that 'need to create new translated-name IDs in
versionning code for startup file' situation is really seldom.

Anyway, hopefully that will fix the crash.

Prevent G.fileflags changes when WM_OT_save_mainfile() is called from script

This is to solve an issue where a blend file could be compressed
unbeknownst to the artist. This happened in the following situtation:

- Artist edits an uncompressed blend file.
- Some script saves a compressed blendfile to a separate location.
- When the artist saves the file (s)he is editing (File>Save, or Ctrl+S),
  it was silently compressed.

Cherry pick from: cd3b313d5f44a10a1150bf1ddb560775d1bcd827

Fix T57328: 'Save As Copy' prints warning

Cycles: Enable shadow information in requested features when denoising is used

The shadow information isn't only used for shadowcatchers, but also to generate the shadowing feature pass of the denoiser.

build_depenencies: Fix url for openal.

Fix T57284: poor result UV unwrapping concave n-gon.

Fix error disabling all addons

Fix Copy Selection to Buffer breaking library paths.

This is a bug experienced by animators in the Blender Studio that developers
have been trying to fix for a /long/ time.

What happens is that partial file writing extracts the needed datablocks from
the main list of datablocks into a smaller one. Afterwards they are added back
to the main list, but in some cases not exactly in the same order.

There is file path remapping code that depends on the datablocks being in
exactly the same order as before, and when this was not the case filepaths
would get swapped between datablocks

The reason datablocks are not restored in the same order is because the sorting
of datablocks by name is a) case insensitive and b) undefined if there are
multiple datablocks with the same name from different libraries. This should
be made well defined, but the fix in this commit is simpler.

The way animators ran into this bug is that they use the Copy Attributes addon
a lot, which has as the first item in the menu Copy Selection to Buffer. In
some cases this would be clicked accidentally when menu is near the edge of the
window, breaking the library paths which would only be noticed a much later on
file save and reload.

The way this bug was finally tracked down is that it was suspected that the
undo system was the cause, and so Bastien added library validation for undo.
When Hjalti then did undo and noticed the error, he remembered accidentally
clicking Copy Selection to Buffer just before, and we could finally reproduce
the bug.

DEBUG_IO: add sanity check on libdata in undo step as well.

There are serious suspicions that weird corruptions faced by studio
artists may happen in undo/redo code, so let's see whether that's the
case.

With this, and when --debug-io arg is passed on startup, the whole lib
data are checked at every undo. This makes undo slower (from two to
three times slower), but it could help us spot better what happens...

Expose 'debug_io' flag in bpy.app

Minor style cleanup.

Fix cmake not triggering rebuild on .glsl changes

At least on windows we do not re-run datatoc when the .glsl files change.

To test is simple, just change edit_mesh_overlay_common_lib.glsl
remove lines, write plain text, ..., now rebuild and go in edit mode
with the default cube.

I also had to remove the entry in gpu/CMakeLists.txt for
gpu_shader_material.glsl since this was being tracked directly, as well
as running data_to_c_simple (otherwise CMake raises an error for
duplicated entries).

We probably want to do the same for the other datatoc functions.

Reviewers: LazyDodo, brecht
Differential Revision: https://developer.blender.org/D3803

Fix incorrect strncpy use

Didn't ensure null terminated.

Cleanup: style (pointers)

Cleanup: spelling

Cycles: Fix NLM denoising kernels zeroing the wrong buffer on OpenCL

Since my temporary buffer commit (about a month ago), the OpenCL device was zeroing the wrong buffer, leading to
completely wrong filtered feature passes and therefore significantly lower-quality results than CPU and CUDA.

Cleanup: style

Fix T57103: Subdivide smooth results in NAN verts

Cycles: Clean up extra minus in previous commit

Forgot to add that change, sorry for the noise.

Cycles: Reuse existing buffer in the NLM denoising kernels on CPU

Cycles: Use existing shared temporary memory in reconstruction step of the denoiser

Previously the code allocated its own temporary memory, but it's possible to just use the existing shared one instead.

Tests: Support parallel execution of render tests

Previously, parallel tests would overwrite each others temporary outputs.

Cycles: Implement vectorized NLM kernels for faster CPU denoising

Fix T57059: "make deps" build error when sndio library is present.

cycles: Support building with cuda 10 on windows.

Fix build with older CMake versions that we still need to support.

Fix T54287: memory not freed after rendering on Linux.

With new jemalloc versions memory allocated by threads that then become
inactive is not longer automatically freed. Instead we have to enable a
background thread to do it.

Some testing is needed to find out of this is sufficient, because the
background thread only runs periodically.

Correct tooltip

Fix T56909: wrong interface scale on macOS 10.14, when using 10.14 SDK.

This does not affect existing releases as far as I can tell, only new builds
using the new SDK have the problem.

BLI_math: add `isect_seg_seg_v3`

Fix T56396: Cycles wrong object motion blur with deformation blur disabled.

Cycles: Fix compilation error of CUDA kernel

Was caused by previous commit.

Tests: recurse into directories for finding OpenGL draw test .blends.

Cycles: Fix crash with BVH8 on certain scenes

The crash was caused by BVH traversal stack being overflowed.

That overflow was caused by lots of false-positive intersections
for rays originating on a non-finite location.

Not sure why those rays will be existing in the first place,
this is to be investigated separately.

This commit moves pre-SSE4.1 check to a higher level function
and enables it for all miroarchitectures.

Fix T56905: unsupported channel layout error writing AAC audio.

This uses same mapping as Audaspace to specify channel layout, which was
missing before.

Fix compilation errors and warnings with Clang-6

Fix cycles_cubin_cc build error on macOS.

Spelling fixes in comments and descriptions, patch by luzpaz.

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

Fix CUDA build with Xcode 10.0, use nvrtc due to incompatible compilers.

Cycles: Cleanup

Cycles: Sync BVH8 unaligned node packing code with BVH4

Similar to dfae3de6bdf.

Tests: add --no-window-focus to open window without focus.

This is useful to run OpenGL tests while continuing to do other tasks
without windows constantly popping up in the foreground.

Spin Tool: hide options unrelated to duplication

Cleanup: redundant include

BMesh: spin option to merge first/last

For 360d revolutions this is almost always whats intended,
enable by default.

CMake: Repress deprecation warnings with MSVC.

Fix UV edit smart stitch not redoing correctly.

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

Fix missing metaball capsule properties in properties editor.

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

Fix T56912: bpy.data.masks.new() crashed

I think there are two possible ways to fix that.
1. Make the name a required parameter.
2. Provide a default value.

I choosed option 1 in this fix to be consistent with other .new functions.

Also I think `RNA_def_string` instead of `RNA_def_string_file_path` should be used here. Looks like a copy-paste error.

Reviewers: brecht

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

Missing from last commit

BMesh: option to flip normals for extrude/spin

Cleanup: style

Fix T56861: freestyle + BI + full sample AA not working correct.

Spelling fixes in comments and descriptions, patch by luzpaz.

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

Tracking: Use pixel aspect from clip

Don't force square pixel, since on a more real pipeline
expected delivery is same anamorphic as an input footage.

Fix T56875: Auto Render not working after render pass name refactor

The default compositor node update function sets the need_exec flag on the node
which the Auto Render feature checks, but the custom update function that was
added as part of rB4cf7fc3b3a4d didn't do so.

Therefore, the two custom update functions that were added now also call the
default update function.

Cleanup: style, use raw strings

Sculpt: Fix memory leak in undo system

Was not freeing undo nodes themselves.

Cleanup: use win_to_3d_on_plane for curve drawing

3D View: add clip argument to win_to_3d_on_plane

3D View: add a simple un-clipped win_to_ray

Cleanup: rename 3D view ray calculation functions

Using near far and optionally clipping planes is
involved and not needed in many cases.

Rename so a simpler version of this function can be added.

3D View: point-on-plane from screen location utility

Cleanup: style

Fix T53745: Alembic exported with vertex colors not readable by other software.

Most other software expects to read indexed vertex colors, so write indices
along with the colors as we already do for UVs.

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

Cycles: Cleanup, double semicolon

Spelling fixes in comments and descriptions, patch by luzpaz.

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

make.bat: fix build error when there are spaces in the path to svn/git/cmake

Fix wrong CUDA version warning in cmake.

Fix suggested by Dalai.

Fix possible misuse of `BLI_strncpy`.

Same issue as rB39f7c8256d58.

Fix unused var in case Alembic is not enabled, in own previous commit.

Sorry for the noise...