Blender Git Loki

Cleanup: Else after return in Cycles

Cleanup: Explicit specifier for single argument constructor

Fix T91190: Remove gaps operator not working

Caused by mistake in f49d438ced7c - `SEQ_query_all_strips()` is not
meant to be recursive.

Sculpt: fix sharp brush, new experimental
mode was being enabled when it
shouldn't be

Sequencer: hide gizmos & cursor during scrubbing & playback

This was distracting prevented easily viewing an animation.

Merge remote-tracking branch 'origin/master' into temp-lineart-contained

Fix crash switching scenes with sequencer gizmos displayed

Sequencer: add option to toggle gizmos

Use shortcut matching the 3D view & popover in the header

Sculpt: fix broken jitter and smooth
stabilize brush settings.

Cleanup: spelling

Sequencer: only show the 2D cursor with overlays enabled

Also hide when displaying scopes.

Fix sequencer preview/strip checks

- Drawing annotations used a deprecated value to detect
if drawing was supported.
- ED_space_sequencer_check_show_strip was checking the preview
image type (which doesn't impact strip display).

Signed-off-by: Campbell Barton <ideasman42@gmail.com>

Cleanup: remove deprecated SEQ_DRAW_SEQUENCE value

While drawing cleared this value (as part of temporary fix from 2009),
this was still being checked until recently.

Remove this value in versioning code.

Also clear unused text space flag.

Cleanup: remove redundant cursor copy, print

Cleanup: Use `_` suffix for non-public members in Compositor

To follow the style guide.

Cleanup: Convert camelCase naming to snake_case in Compositor

To follow the style guide.

Cleanup: Remove unused includes in Compositor

And move unneeded includes in frequently used headers
to source files.

Slightly reduces compilation time.

Cleanup: Remove `this->` for `m_` prefixed members in Compositor

The prefix is already explicit.

Cleanup: remove unnecessary data from LocalIntersection

Fix Cycles random walk SSS differences between CPU and GPU

The Embree logic did not match the GPU.