Fix (unreported) usercount of linked IDs becoming garbage after undo/redo.
authorBastien Montagne <montagne29@wanadoo.fr>
Wed, 22 May 2019 19:10:36 +0000 (21:10 +0200)
committerBastien Montagne <montagne29@wanadoo.fr>
Wed, 22 May 2019 19:10:36 +0000 (21:10 +0200)
commit3a702ec028231e7deb012edac3eee5ed7f2436d4
tree6be0f279d128d5f204ef89b4b8c1b20efdb6d41e
parent3600e94ebac0d411042c54ed0970099a7a9cb932
Fix (unreported) usercount of linked IDs becoming garbage after undo/redo.

Not re-reading linked data-blocks in undo/redo case also means that we
do not touch to their usercounts. Even worse, lib_link process in
readfile will increase those (for cases where local data uses linked
one).

Whole data management code is now heavily relying on valid consistent
refcount of all IDs, so we cannot allow that anymore.

Simple solution here could have been to then not increase that one for
linked IDs in `newlibadr_us()`, but unfortunately that would not be
totally bullet-proof, as some local users of linked data may be added or
removed by an undo step...

So I cannot think of any other solution than the ugly brute force one,
i.e. going over the whole Main database and recompute linked IDs users
count... Should not be a big issue performance wise though, this is
fairly cheap process.
source/blender/blenloader/intern/readfile.c