Blender Git Loki

Merge branch 'master' into blender2.8

Cleanup: rename 3D cursor calculation vars

Cleanup: 2.8-new G.main in RNA code.

Again, essentially validating them, and adding assert check if needed.

Merge branch 'master' into blender2.8

Conflicts:
source/blender/makesrna/intern/rna_image.c
source/blender/makesrna/intern/rna_object.c
source/blender/makesrna/intern/rna_object_force.c
source/blender/makesrna/intern/rna_screen.c
source/blender/makesrna/intern/rna_sculpt_paint.c
source/blender/makesrna/intern/rna_space.c
source/blender/python/bmesh/bmesh_py_types.c
source/blender/python/generic/bpy_internal_import.h
source/blender/python/intern/bpy_rna_anim.c
source/blender/python/intern/gpu_offscreen.c

StudioLight: Only apply windowing when SH level is 2

It does not make much sense for the other levels

Cleanup: remove last G.main from RNA.

Again, we cannot actually get rid of G_MAIN global access here, so in
most case just 'marked' them as valid, and added assert checks to ensure
we do only work with IDs in G_MAIN in those cases.

StudioLight: remove caches when removing studiolight

Cache files were not deleted and when uploading a new file with the same
name resulted in using the old cache file.

Studiolight: Spherical Harmonics Windowing

Apply Windowing on the Spherical Harmonics result. This would lead to
better results.

Tweak new BKE_id_is_in_global_main to accept NULL pointer (and consider them as valid).

Fix harmless use of unintialized memory

Still nbice to avoid such access to keep valgrind output more sane.

Cleanup, spelling

Math: Set matrix to zero when inversion fails

Avoids usage of uninitialized memory when inversion fails.

That uninitialized memory can cause object to become visible when
it is supposed not to or other artifacts like that.

Longer term solution would be to check every instance of invert_m#
function and to explicit fallback when needed (possibly, using
extra utility functions).

Cleanup: get rid of last G.main's in bpy area.

Essentially 'validating' them as G_MAIN, and adding some asserts in
dubious places, that handled IDs are actually in G_MAIN.

BKE_library: Add func to check an ID is actually in G_MAIN database.

Uncrustify processed.

Offline intersection and occlusion test code complete.

PaintMode: Full Shading Boolean => Slider

There was a Full Shading bool that was shared across the WP, VP and TP
modes. This commit makes some changes:

- Replace the bool with a factor. This gives the user more control on
the visibility.
- Also draw it on top of the Material and Rendered mode so the user can
control what he needs. In certain cases you don't want to see the final
rendered material, but the actual texture.
- Removed the skipping of objects when in paint modes. As now the paint
modes are blended.

Fix reading modifier object pointers

Kudos to Bastien Montagne to help figure out the special modifier read code.

Remove unused variable

(instead of poorly silencing it with UNSED =))

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