Merge branch 'master' into blender2.8

Make GS macro return proper IDType type

Previously it was returning short, which was really easy to (a) compare against
non-ID type value (b) forget to handle some specific value in switch statement.

Both issues happened in the nearest past, so it's time to tighten some nuts
here.

Most of the change related on silencing strict compiler warning now, but there
is also one tricky aspect: ID_NLA is not in the IDType enum. So there is still
cast to short to handle that switch. If someone has better ideas how to deal
with this please go ahead :)

Atomics: Use system headers directly, without bad level dependency to BLI

This will make it easier to re-use library as-is in other projects,
such as Cycles standalone repo for example.

Depsgraph: Pass copy-on-write pointer to material update

Unfortunately, there is something else wrong going on here, which makes objects
black after tweaking material settings.

Merge branch 'master' into blender2.8

Buildbot: Tweak nightly schedule of builds

Basically, schedule linux 64 earlier on, so builds are ready for nightly benchmarks.

Missing from last commit (missed manually resolving)

Merge branch 'master' into blender2.8

Transform: center override

Hidden option to override transform center.
Needed for manipulators that define their own center.

Merge branch 'master' into blender2.8

EditMesh: set edge-rin subdiv minimum to 1

This only made sense for bridge tool.

D2785 by @mbjorkegren

Docs: BMesh.from_mesh behavior w/ multiple calls

Fix T51400: Pasting hex code fails

The # prefix is supported,
the button didn't give enough space to paste it.

D2812 by @candreacchio

Fix T52498: Deleting force field doesn't remove "Surface" from modifier stack.

Logic in `ED_object_check_force_modifiers` was inconsistent between add
and remove modifier cases.

Should be safe enough for 2.79.

EditMesh: set edge-rin subdiv minimum to 1

This only made sense for bridge tool.

D2785 by @mbjorkegren

Docs: BMesh.from_mesh behavior w/ multiple calls

Fix T51400: Pasting hex code fails

The # prefix is supported,
the button didn't give enough space to paste it.

D2812 by @candreacchio

Fix blenderplayer compilation

Manipulator: support operator per-part

A single manipulator could only assign a single operator to each part.
Now each part can have it's own.

Also modify 2D selection callback, 2D started at 1, 3D at 0.
Now use -1 for unset value, start both at 0.

Fix T52498: Deleting force field doesn't remove "Surface" from modifier stack.

Logic in `ED_object_check_force_modifiers` was inconsistent between add
and remove modifier cases.

Should be safe enough for 2.79.

Merge branch 'master' into blender2.8

BLI_rect: Function to calculate a matrix from 2 rctf's

Missing from last commit

Cleanup: use stubs for eigen gtest

Correct matrix stack assert

Missed last commit

Fix T52515: Crash on BMesh.to_mesh()

Merge branch 'master' into blender2.8

WM: move theme check out of ED_view3d_draw_select_loop

In 2.8x this needs to be in view3d_opengl_select,
so simplest to make in master too.

Fix minor Mesh -> BMesh conversion issues

- Vertex only meshes never restored their selection history.
- Select history was cleared on the source instead of the target.

Simple Optimizations:
- Avoid O(n^2) linked list looping that checked the entire list before
  adding elements (NULL values in the source array to prevent dupes).
- Re-use vert & edge lookup tables instead of allocating new ones.

Fix T52478: Error report "Shrinkwrap: out of memory" on invisible target.

Shrinkwrap must check it does have valid target data.

Safe for 2.79 release.

Fix T52538: Outliner crash when displaying groups and using Show Active on editmode bone not in any groups

There's no guaranty that given ID is found in current outliner tree...

Safe for 2.79, though not a regression.

[cycles/ctest] fix failing tests when output folder doesn't exist yet.

Fix for fix (tm): Residue of the debug code

Fix mistake in previous tangent space optimization

Cycles: Cleanup, naming of variable

Always use b_ prefix for C++ RNA data.

Cycles: Add assert to catch possibly wrong logic

Cycles: Mark pixels with negative values as outliers

If a pixel has negative components, something already went wrong, so the best option is to just ignore it.

Should be good for 2.79.

Fix T52481: After making all local, local proxies of linked data get broken after file save and reload.

Issue was nasty hidden one, the dual status (mix of local and linked)
of proxies striking again.

Here, remapping process was considering obdata pointer of proxies as
indirect usage, hence clearing the 'LIB_TAG_EXTERN' of obdata pointer.
That would make savetoblend code not store any 'lib placeholder' for
obdata data-block, which was hence lost on next file read.

Another (probably better) solution here would be to actually consider
obdata of proxies are fully indirect usage, and simply reassign proxies
from their linked object's obdata on file read...

However, that change shall be safer for now, probably good for 2.79 too.

Revert "material Glsl: Fix tangent with new orco."

This reverts commit 3888227a7bfdb6b3867680ea237152d2929e1bc5.

This "Fix" was made while ORCO was broken. Now that orco itself is fixed
this is no longer required, otherwise Tangent node produces different
results in Cycles and Eevee.

Another optimization of tangent space calculation

Don't use quick sort for small arrays, bubble sort works way faster for small
arrays due to cache coherency. This is what qsort() from libc is doing actually.
We can also experiment unrolling some extra small arrays, for example 3 and 4
element arrays.

This reduces tangent space calculation for dragon from 3.1sec to 2.9sec.

Cleanup: missing break (harmless for now)

Optimize tangent space calculation by inlining functions

Brings tangent space calculation from 4.6sec to 3.1sec for dragon model in BI.
Cycles is also somewhat faster, but it has other bottlenecks.

Funny thing, using simple `static inline` already gives a lot of speedup here.
That's just answering question whether it's OK to leave decision on what to
inline up to a compiler..

Cycles: Add utility function to query current value of scoped timer

Cycles: Correct logging of sued CPU intrisics

Cycles: Fix stack overflow during traversal caused by floating overflow

Would be nice to be able to catch this with assert as well, will see what would
be the best way to do this/.\

Need to verify with Mai that this solves crash for her and maybe consider
porting this to 2.79.

Fix T52528: generated texture mismatch between Cycles and Eevee

Finally orco should be working 100% on Eevee now. Thanks for Sergey
Sharybin for reporting this.

Revert "Eevee: Fix generated coordinates when no texture coordinates connected"

This reverts commit 0e29a97813d2a2df77afe64d49d1683687ed37ab.

Fix ORCO never used for draw manager (Eevee)

Related to T52528.

Merge branch 'master' into blender2.8

WM: ensure 3D view theme is used in select-loop

Yet another case where theme could be used uninitialized.

Merge branch 'master' into blender2.8

WM: initialize WM and deps before handling events

This avoids obscure bugs where operators could run from
events that happen before the UI and depsgraph have been initialized.

See: D2809 for details.

Eevee: Fix generated coordinates when no texture coordinates connected

Orco should behave the same if it comes from unconnected vec inputs, or
from the Texture Coordinate -> Generated node output.

Fixup for 11e7e0769a4c.

This is related to T52528. The coordinates are still different between
Eevee and Cycles, but at least it behaves consistent within itself.

In fact the shader should now be correct, but the orco attributes we are
passing the shader seems to be where the problem is. But it's to be
tackled separately.

This copyright text (copied to binary distros) had a confusing statement about
scripts being "Artwork" which is your sole property and free to license.

I've removed the reference to scripts in this text.

This was from 2002! With our Python scripts becoming part of how Blender runs,
such scripts now are officially required to be compliant with GNU GPL.

For more information; check the FAQ or consult foundation@blender.org
https://www.blender.org/support/faq/

Fix manipulator remove tag, it cleared all groups

Manipulator: zero grab offset w/o target property

Some manipulators are used like on-screen buttons,
in this case it doesn't make sense to keep track of their state,
so zero the offset when its unused.

Needed for lamp-target manipulator.

Correct error in recent use of PyC_Long_*

Regression in 46cf33bf0

Fix crash handling manipulator events before drawing

Depsgraph: Fix crash editing mesh in edit mode

Depsgraph: Cleanup typo

Fix error using wrong theme settings w/ depth draw

Clicks events before first draw would assert,
unlikely to cause user visible errors but
would assert in UI_ThemeGetColorPtr.

Fix T51907: New Depsgraph - Camera constraint is not evaluated properly

This is more a workaround for until we've got proper visibility flush, which
will likely happen in blender2.8 branch.

Cycles: FIx issue with -0 being considered a non-finite value

Merge remote-tracking branch 'origin/master' into blender2.8

Cycles Bake: Fix overflow when using hundreds of images

We have a hardcored limit of 1000 images to be baked.
However anything anove 100 would be leading to overflow in the code.

Caught by warning from builder bot (my compiler doesn't even complain
about this, but it should).

Manipulator: modal callback can now cancel & pass events

Re-use operator return flags for manipulator modal & invoke,
this means manipulators can allow navigation or other events to be
handled as they run - see T52499

Manipulator: remove unused event hack

Fix Cycles CUDA transparent shadow error after recent fix in c22b52c.

Fishy cat benchmark was rendering with wrong shadows. Cause is unclear,
adding printf or rearranging code seems to avoid this issue, possibly a
compiler bug. This reverts the fix and solves the OSL bug elsewhere.

Code cleanup: remove shader context.

This was needed when we accessed OSL closure memory after shader evaluation,
which could get overwritten by another shader evaluation. But all closures
are immediatley converted to ShaderClosure now, so no longer needed.

Docs: rename var and comment how it's used

switch_from_camera wasn't right since it was used for auto-perspective.

Fix blenderplayer (tm)

Merge branch 'master' into blender2.8

Correction to last fix

Fix T52490: NDOF orbit doesn't lock in ortho view

Regression in af3f7db caused by own fix for T51324

Fix bplayer (c)

Fix T52396: Crash loading template w/o config dir

Cycles: Add maximum depth stat to bvh builder

Fix implementation of atomic update max and move to a central location

While unlikely to have had any serious effects because of limited use, the
previous implementation was not actually atomic due to a data race and
incorrectly coded CAS loop. We also had duplicates of this code in a few
places, it's now been moved to a single location with all other atomic
operations.

Fix T51805: Overlapping volumes renders incorrect on AMD GPU

We need to make sure we can store all volume closures for all objects in volume
stack. This is a bit tricky to detect what would be the "nestness" level of
volumes so for now use maximum possible stack depth. Might cause some slowdown,
but better to give reliable render output than to fail quickly.

Should be safe for 2.79 after extra eyes.

Merge branch 'master' into blender2.8

Cleanup: mark VA_NARGS_COUNT as public

Was already used in two other headers, remove underscore prefix.

Cleanup: move variadic defines to their own header

So we can use in headers without pulling in many other defines.

Fix T52218: Missing update when reconnecting node

If node was connected to output, we tag tree for update no matter where
the node was re-plugged to.

Should be safe for 2.79.

RNA: use string join functions as with operators

Merge branch 'master' into blender2.8

RNA: use string-join to simplify operator register

Also sanity check macro-operator ID's.

BLI_string_utils: string joining utility functions

Includes a version that takes a separator and macros for convenience.

PyAPI: avoid instantiating args twice in macro

Would cause problems if args included function calls.

RNA: check for duplicate manipulator names

Merge branch 'master' into blender2.8

Cleanup: remove space from filenames

Tests: fix incorrect check for hidden dir

Copy-pasted mistake in tests and tools.

RNA: report error on struct naming collision

Fixes T52463, error instead of crash.

RNA: keep structs_map valid w/ ID duplicate & free

GHash: BLI_ghash_reinsert_key utility function

Useful when ghash keys are reallocated.

Fix T52466: Silence search for button_context menu type.

We were showing "search for unknown menutype WM_MT_button_context" messages in terminal which were not helpful for users, so now they are disabled.

To be backported to 2.79

Fix error in PointerProperty argument list

Regression in a7b3047

Fix T52483: Fill is incorrect for interpolated strokes

The recalc flag must be enabled for new interpolated strokes.

Merge branch 'master' into blender2.8