diff options
author | Sergey Sharybin <sergey.vfx@gmail.com> | 2013-12-16 20:05:27 +0400 |
---|---|---|
committer | Sergey Sharybin <sergey.vfx@gmail.com> | 2013-12-16 20:05:27 +0400 |
commit | 2231b565f011e984d4ee6eeb33a80567b404e032 (patch) | |
tree | e23181cc9571029f27199d44b178441291fd8ebe /source/blender/imbuf/intern/moviecache.c | |
parent | 28d873f038ca67dd267e8939e5a32eedeb23cba8 (diff) |
Better fix for previous commit
Cache iterator might return unused keys as well.
Now unused keys are being removed before cleanup
and iteration.
Diffstat (limited to 'source/blender/imbuf/intern/moviecache.c')
-rw-r--r-- | source/blender/imbuf/intern/moviecache.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/source/blender/imbuf/intern/moviecache.c b/source/blender/imbuf/intern/moviecache.c index 20e0169658f..c042831a93f 100644 --- a/source/blender/imbuf/intern/moviecache.c +++ b/source/blender/imbuf/intern/moviecache.c @@ -447,6 +447,8 @@ void IMB_moviecache_cleanup(MovieCache *cache, bool (cleanup_check_cb) (ImBuf *i { GHashIterator *iter; + check_unused_keys(cache); + iter = BLI_ghashIterator_new(cache->hash); while (!BLI_ghashIterator_done(iter)) { MovieCacheKey *key = BLI_ghashIterator_getKey(iter); @@ -454,7 +456,7 @@ void IMB_moviecache_cleanup(MovieCache *cache, bool (cleanup_check_cb) (ImBuf *i BLI_ghashIterator_step(iter); - if (item->ibuf == NULL || cleanup_check_cb(item->ibuf, key->userkey, userdata)) { + if (cleanup_check_cb(item->ibuf, key->userkey, userdata)) { PRINT("%s: cache '%s' remove item %p\n", __func__, cache->name, item); BLI_ghash_remove(cache->hash, key, moviecache_keyfree, moviecache_valfree); @@ -554,7 +556,11 @@ void IMB_moviecache_get_cache_segments(MovieCache *cache, int proxy, int render_ struct MovieCacheIter *IMB_moviecacheIter_new(MovieCache *cache) { - GHashIterator *iter = BLI_ghashIterator_new(cache->hash); + GHashIterator *iter; + + check_unused_keys(cache); + iter = BLI_ghashIterator_new(cache->hash); + return (struct MovieCacheIter *) iter; } |