Blender Git Loki

Readfile: refactor/factorize more handling of common ID data.

- Move handling of private ID data (nodetree and master_collection)
under generic ID code. This shortens code a bit, but mostly avoids having
to modify all type-specific callback functions if/when we have to add
generic processing to IDs there.

- Seriously factorize `expand_xxx` area, in the same way we were already
doing in `direct_link_xxx` and `lib_link_xxx` areas.

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

NodeTree: Add access to the address of an ID's nodetree pointer.

Merge branch 'master' into undo-experiments

Better handling of usages of usercount increment in readfile liblink.

Some places could/should still need proper ID refcounting in liblink:
* UI data (spaces) - although this should be strictly 'user one' only...
* Deprectaed data/pointers which are not handled by libquery code.

Not sure whether IPO pointers really need usercount actually...

Merge branch 'master' into readfile-rework-refcount-handling

Conflicts:
source/blender/blenloader/intern/readfile.c

readfile: Quick experiment with refcounting moved to libquery.

Having that extra ID users handling at readfile level, besides generic
one ensured by libquery, has been something bothering me for a long time
(had to fix my share of bugs due to mismatches between those two areas).

Further more, work on undo speedup will require even more complex ID
refcount management if we want to keep it in readfile.c area.

So idea is instead to generalize what we did for linked data already
when undoing: recompute properly usercount numbers after liblink step,
for all IDs.

Note that extra time required here is neglectable in a whole .blend file
reading (few extra milliseconds when loading a full production scene
e.g.).

Merge branch 'master' into undo-experiments

Refactor readfile's liblink code.

Liblink specific ID type function was so far running a loop over all IDs
of relevant type, unlike almost any other 'ID-callback-like' functions
in Blender, which usually let the looping controll to calling code.

The latter approach is more convinient when one want to add generic
(i.e. type-agnostic) code, since it typically only has to change code in
one place (caller function) instead of tens of places (all the callback
functions).

This commit also changes/sanitizes a few things that had nothing to do
in main liblink code, like mesh conversion from tessfaces to polys
(which can be done in after-linking versionning code), or scenes' cycles
detection/check regarding background 'set' scenes.

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

undoexp: cleanup: move/tweak assert re undo direction value.

Merge branch 'master' into undo-experiments

undoexp: Fix issue with usual infamous pose's bone pointers.

undoexp: cleanup typo.

undoexp: tweak `setup_app_data()` to better handle undo case.

The way we detected undo case there was somewhat weak, now we have a
stronger way to do it from `BlendFileReadParams`.

undoexp: fix memleak after recent change of WM/SCR/WS IDs handling.

Merge branch 'master' into undo-experiments

undoexp: fix bug/crash in recent refactor of WM/SCR/WS IDs handling.

Mismatch when partial undo was not enabled.

undoexp: cleanup/simplify handling of WM/SCR/WS IDs.

Re-read from memfile WindowManagers, Screens and WorkSpaces
are never actually used in undo situation. This allows us to simply
early abort on those ID types, even if we do not find them in current
(old) bmain for some reason.

undoexp: style cleanup.

Merge branch 'master' into undo-experiments

Merge branch 'master' into undo-experiments