Blender Git Loki

Fix T81200: Crash on certain armature select operators when OpenGL Depth
Picking is disabled

Affected were (at least)
- Select Linked pick
- Select Shortest Path

Was originally caused by rB8b347fc2cdc6 [which got improved in
rBfc8a7a44b200 - but only partially].

After removal of BONESEL_NOSEL we have to filter out -1 from the hits for
any bone picking it seems (since armature drawing uses this).
In rBfc8a7a44b200, this was done for 'ED_armature_edit_select_pick' (or
'get_nearest_editbonepoint' precisely), now do this for
'ed_armature_pick_bone_impl' as well.

Reviewers: campbellbarton

Maniphest Tasks: T81200

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

Gpencil: location calculation changed to obmat

UI: Improve UV sticky select mode description

The sticky_select_mode property is an enum where each item has a
different behavior, but currently the description is for a single one
of the items. It should be more general, especially because one of the
items is "Disabled."

A couple more problems:
- "Automatically" is not so helpful. Many things any program does are
"automatic", it doesn't really help to say it.
- "Also" is bad grammar.
- Overly complex wording

This also adjusts the SHARED_VERTEX tooltip, because "irrespective"
should have "of" after, but it's also unecessarily complex.

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

Merge remote-tracking branch 'origin/master' into gpencil-fading-modifier

GPencil: property name change to "use_fading".

Cleanup: Refactor `ed_screen_context()` to use hash lookups

Refactor `ed_screen_context()` to use `GHash` lookups instead of a
sequence of string comparisons. This should provide a nice speedup,
given that the hash for `member` only has to be computed once instead of
matching it to each possible string.

Reviwed by: brecht

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

Cleanup: split up `ed_screen_context()` into separate functions

Refactor `ed_screen_context()` to call separate functions, instead of
having the entire functionality in one function. Each function now only
retrieves the data it needs from the context. Furthermore, some string
comparisons are removed.

No functional changes.

Reviwed by: brecht

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

Volumes: make Mesh to Volume modifier independent of object transforms

Experimental Features: More robust way of sanitizing options

Experimental Features: Sanitize missing (latest) options

There was an oversight when adding new experimental user preferences.
I can try to overengineer this later to make it more fail-proof. But for now
it should be clear what to update when adding a new variable.

Experimental Features: Rename Image editor option

This prevents having multiple UI strings starting exactly the same:
* Legacy A...
* Legacy B ...

Merge remote-tracking branch 'origin/master' into gpencil-fading-modifier

GPencil: Fading functions in opacity and thickness modifiers.

CleanUp: Remove header only definition

node_draw_link_straight existed in header files without a body

Cleanup: reduce variable scopes

Merge branch 'master' into soc-2020-io-performance

Cleanup: remove redundant theme versioning

All themes older than 2.8x are initialized from the defaults,
there is no need to version them.

Merge branch 'asset-metadata' into asset-browser

Cleanup: spelling

Also correct own correction from 58b8724a4892
thanks @mont29 for raising this.

Fix userpref.blend using version from startup.blend

Version patching userpref.blend wasn't using the correct version,
causing settings not to be properly updated.

This seems the likely cause of T70196 and similar bugs.