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...

Mesh Sequence Cash: do not *always* report as depending on time!

Fix unreported: Undo of texts pasted in the Text Editor misses one character.

Caused by commit rB28c34ae7e2d6
Unlike `strncpy`, `BLI_strncpy` ensures the last character is '\0'.

BLI_utildefines: rename pointer conversion macros

Terms get/set don't make much sense when casting values.

Name macros so the conversion is obvious,
use common prefix for easier completion.

- GET_INT_FROM_POINTER  -> POINTER_AS_INT
- SET_INT_IN_POINTER    -> POINTER_FROM_INT
- GET_UINT_FROM_POINTER -> POINTER_AS_UINT
- SET_UINT_IN_POINTER   -> POINTER_FROM_UINT

Cleanup: trailing space

Fix T56833: "zoom to cursor" in Clip editor not handling aspect ratio.

Trivial fix, just using same code as in Image editor...

Close OpenEXR thread pool on exit

This partially solves ASAN report about unfreed memory. There is still
something in the report, need to have a closer look with debug version
of OpenEXE library.

FIX: linker error with a 'make deps' library set and opencollada enabled.

PCRE_LIBRARIES was not being set if the pcre library was found but not the headers.

Cycles: Fix usage of AVX2 intrinsics in AVX kernel

While building the AVX kernel, util_avxf.h/avxb.h were using some AVX2 intrinsics,
these were never called, so it wasn't a run-time issue, but the intrinsics headers
on centos excluded the AVX2 prototypes when building the AVX kernel causing build errors.

This commit cleans up the improper usage of the AVX2 intrinsics and provides AVX
fallback implementations for future use.

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

build_files: bump python version in FindPythonLibsUnix.cmake to 3.7 now that all platforms have been updated

build_environment: linux/python, help python find ffi.

we were building ffi, but python wasn't finding it.

build_environment: ffi/linux force predictable lib path.

ffi stubbornly wants to put libs in lib64 even when you tell it not to on some linux distributions.
patch based on sed fix the gentoo guys did [1]

[1] https://bugs.gentoo.org/462814

Install_deps: workaround building bloody broken OpenEXR release.

Cycles: Cleanup, remove dead code

Cycles: Remove unused malformed function

This isn't really possible to do the shuffle which was attempted to do.

While it's possible to achieve expected behavior, the function needs to
be rewritten. Since it's not used anyway, it's simpler to remove it for
now.

Cycles: Use proper mask for vectrorized boolean print

build_environment: force a consistent libdir across Linux distributions.

on some distributions libs ended up in lib64 confusing the cmake builder.

Fix object selection with eyerdropper not respecting property poll function.

Fix T56811: Do not show cancel button for jobs when UI is locked.

Since that button is then totally useless and unusable...

Fix OpenColorIO link errors in some Linux builds.

The library can get installed in a lib/static subdirectory, so search there
as well now.

Fix building deps when OpenCV dev package is present.

Since we are not linking against OpenCV ourselves, that generated
linking errors later on (while building OSL e.g.).
Those 'open' libs link against way too many other libs... :/

Thanks to @intrah for initial report (T56785), and @LazyDodo for
suggested solution.

WM: move mousemove out of internal undo function

This causes a feedback loop in 2.8x,
where gizmo redo caused fake mousemove that executed gizmo again.

Move the mousemove into the undo/redo operator.

PyAPI: remove operator methods that leak memory

PyAPI: use brief description for operator repr

Only include description in docstring.

Also avoid using op_get_rna.

Tentative fix for T56770: Crash after set language to Simplified Chinese.

That bug probably did not affect 2.7x, only 2.8 with COW copying IDs in
threads... But root of the issue is that underlying boost i18n lib does
not support well multi-threaded access. So simply forbid any translation
from non-main thread. This *may* be an annoying limit at some point, but
doubt it will be any issue currently.

PyAPI: add API call to get an operators type

Getting the instance leaks memory and was only meant to be used for
generating docs.

Cleanup: _bpy.ops utility to lookup an operator

Cleanup: use PyImport_GetModuleDict

Replace direct access using PyThreadState_GET

PyDoc: replace use of deprecated API call

Cleanup: move 'make help_features' into own file

PyDoc: disable ASAN exit code for doc building

Prevented docs from building on any minor issue when ASAN was enabled.

Cleanup: GCC ignored qualifier warning

Fix T56662: Autocomplete for texture slot Crash (in console).

That pointer can be NULL, RNA default string handling does not support
that. (that whole uv_layer prop is quite nasty actually, since it does
not own that string, always borrows it from some other data :((( ).

Cycles: Don't allocate Extra if BSDF allocation failed

Failed as in did not allocate due to possibly weight cutoff.
Tryign to allocated Extra storage for closure in such situation
will consfuse Cycles and cause crashes later one due to obscure
values in ShaderData.

Cleanup: More correct terminology work

Matches threading API.

CMake: add openjpeg 2x search paths

Wasn't finding the include path by on Linux.

Cleanup: remove alpha_check variable from DNA

Just pass as an argument.

Build: require OpenJPEG 2.x minimum, remove bundled version.

* WITH_SYSTEM_OPENJPEG is removed and is now always on, this was already
  the case for macOS and Windows.
* This should not break existing Linx builds. If there is no new enough
  OpenJPEG installed, CMake will no find libopenjp2 and WITH_IMAGE_OPENJPEG
  will be disabled.
* install_deps.sh was updated with new package names, since distributions
  put this version in a new package.

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

Cleanup: unused variables

Build: update for new macOS libraries.

Run "make update", or "svn update" in lib/darwin to get the matching
precompiled libraries.

Build Environment: updates for macOS.

Mainly this is following Linux to build own xml2/lzma/ssl/sqlite and linking
them all statically. This ensures the Python ssl module uses a recent openssl
version rather than a very old one shipped with macOS.

Correct own error in recent transform orientation changes

UI: split theme draw style into separate field

Was using UI_BLOCK_LOOP to control draw style,
this meant we couldn't use popup theme colors for cases
where it the interface has the same purpose as a popup but happens
not to use this flag.

Fix Cycles principled BSDF black with transmission and IOR 1.0.

Fix T56720: Lattice: crash on setting points_u/v/w from python.

One need to check that there is something to free, before trying to free it...

Cleanup: move area API into own function

Install_deps: fix llvm/clang ARCH package names.

Issue raised in T56540, thanks.

Also tried to re-enable using fedora's llvm/clang packages, would hope
and assume old issue here is fixed.

Install_deps: Fix wrong URL for ILMBase/OpenEXR source archives.

They also went to github...

Install_deps: Fix building python on fedora 28

Looks like libffi is now a dependency of python as well...

Done it for the other distro as well, just in case. ;)

Fix normal transform orientation calculation

When using the 'normal' orientation, the normal would be ignored
if the plane couldn't be calculated.

Now use only the normal if the plane is zero length,
this was already done, just not in all cases.

Cleanup: fix compiler warning.

Fix T56704: black / NaN values in Cycles normal pass.

Fix T56698: saving render result from compositing as EXR is empty.

Update install_deps.sh to latest libs.

Tested with Debian Testing, might need some adjustements for other
distributions...

Also removed last patches we used here, we shall not need any anymore!

Compositor: Always use AA on masks

In a real world it is very weird to disable AA on a mask,
it will give ugly looking result. For some fast preview
passes (like in the node preview) the system can decide
to disable AA without asking user to do anything.

One thing we can consider doing is to remove Feather
option as well. If real compo becomes measurably slower
in cases when mask has no real feather, we can disable
feather internally, without user input. Disabling
feather in the interface is like making things faster
but giving a wrong result, which doesn't sound that
helpful either.

Reviewers: brecht

Reviewed By: brecht

Subscribers: hype, sebastian_k

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

Fix ruler angle arc displaying past endpoints

Cycles: Allow cancel during displacement of meshes

CCG: Remove number of layers from key

The issue there was that number of layers did not include normals,
while element size counts bytes used by normals. This sounds very
fragile and dangerous to work further. Also, one value can easily
be delivered from another, so it is redundancy going on here.

Possible difference now is that multires subdivision will copy
normals to a higher levels. Shouldn't be big of a problem, since
leaving old normals and updating coordinates is not correct either.

Cleanup: replace doxy @ with backslash

The rest of Blender uses backslashes.

Cleanup: style

Fix T54152: --env-system-scripts fails on win32

Fix VSE cut both-sides option

Was ignoring the option, using the mouse in all cases.

D3671 by @ISS w/ edits.

cycles: Fix x86 build error.

WITH_CYCLES_CUDA_BINARIES was set as a regular variable causing it to be overwritten by the release.cmake settings.

Cleanup: use const arg for BLI_rect inside check

UI: cleanup tooltip bounds clamping

Wasn't properly clamping on the window minimum.

ClosthCollision: fully avoid computing BVHTree when we have no collision objects to test against.

Followup to rBc6bbe6c5aac29, much more elegant solution to the problem. ;)

Spelling fixes in comments and descriptions, patch by luzpaz.

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

Fix (unreported) potentially giant memory leak in Cloth collision solver.

Not freeing its BVHTree in case there were no collision objects,
could quickly lead to hundreds of MB of memleak!

Fix Cycles half float pragma for strict OpenCL compilers (like ROCm).

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

PyAPI: add optional imports to expression eval API

Avoids having to use `__import__` to access modules.

Cleanup: move keymap helpers into own file

Cleanup: comment blocks

Buildbot: Usable PIE

Makes it impossible to run from GUI on certain platforms, see T56628.

Python: Enable MASS unit in FloatProperty

MASS unit was already implemented for the C api. Only making sure it is
accessible in the python api. Also added 'CAMERA' to the documentation as a valid option.

Remove CCGDM from PBVH

The statement that PBVH needs to keep track of CCGDM is wrong, PBVH itself
does not care about CCGDM at all, and it's weird for it to carry on this
beast so others can access.

Even more, nobody will actually caring about CCGDM itself, all the usages
were checking whether there is CCGDM or not. This is as good as simply
checking PBVH type.

Tested with an original report T53551 and everything is still stable.

Fix T56618: crash in release builds when GHOST asserts.

WITH_ASSERT_ABORT was not disabled for release builds. In most cases asserts
are disabled in release builds, but not always.

This also changes the buildbot to use blender_release.cmake instead of
blender_full.cmake, the only effective difference should be WITH_ASSERT_ABORT.

Fix T56618: crash when typing certain characters on Linux.

Cycles: Cleanup, use explicit comparison with NULL

Fix T56626: Cycles ambient occlusion only local : crash

Was caused by missing NULL pointer check in BVH8.

PyAPI: replace checks for invalid input w/ assert

Was returning -1 as a bool argument,
in this case the caller needs to ensure non-null args.

C/Python API: Add PyC_RunString_AsIntPtr

Utility to get an int or pointer from a Python expression.

Cleanup: rename WM_keymap_find -> WM_keymap_ensure

Cycles: Cleanup, whitespace after keyword

Buildbot: Correction to previous commit

Buildbot: Attempt to fix glibc-2.19 compilation

It is likely to be retired, but for until then lets ensure it works.

Cycles: Fix detection of CPU brand string on 32 bit platforms

The assembler template was backing up and restoring ebx, which is
fair enough. However, this did not prevent compiler for putting
result variables to ebx. This was causing data corruption.

In order to prevent this easiest solution is to list ebx in clobbers
for the assembly.

Fix T56612: crash in Cycles viewport render update, after recent changes.

BVH8 refitting code had a bug.

Fix T56610: crash saving images with python API.

Cleanup: license header formatting.

Build environment: Tweak to harvasting script

The idea is to make find_package() to succeed without any
extra tips given. For this, we need header files to be
harvasted.

Buildbot: Get rid of OpenJPEG 2 hack

Also clarified some options and decisions.

CMake: Automatically detect OpenJPEG 2 libraries

There is chance that on a system with both versions installed this
*might* cause some issues. Such system will be pain to support out
of the box anyway.

This change allows to use precompiled libraries without extra
modifications in the config.