Revision ea0dc96 by Bastien Montagne (id-ensure-unique-memory-address, readfile-rework-refcount-handling, undo-experiments, undo-experiments-swap-reread-datablocks, uuid-undo-experiments, uuid-undo-experiments-swap-reread-datablocks) February 5, 2020, 16:04 (GMT) |
Merge branch 'master' into readfile-rework-refcount-handling Conflicts: source/blender/blenloader/intern/readfile.c |
Revision 8c75037 by Bastien Montagne (id-ensure-unique-memory-address, readfile-rework-refcount-handling, undo-experiments, undo-experiments-swap-reread-datablocks, uuid-undo-experiments, uuid-undo-experiments-swap-reread-datablocks) February 3, 2020, 09:52 (GMT) |
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.). |
Revision 9829a3c by Bastien Montagne (id-ensure-unique-memory-address, readfile-rework-refcount-handling, undo-experiments, undo-experiments-swap-reread-datablocks, uuid-undo-experiments, uuid-undo-experiments-swap-reread-datablocks) February 1, 2020, 18:43 (GMT) |
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 |
|