Alembic: Moved a BLI_assert
authorSybren A. Stüvel <sybren@stuvel.eu>
Fri, 8 Jun 2018 13:42:51 +0000 (15:42 +0200)
committerSybren A. Stüvel <sybren@stuvel.eu>
Fri, 8 Jun 2018 13:51:49 +0000 (15:51 +0200)
The assert should only be triggered when assigning a pointer, and not
simply when checking the pointer.

source/blender/blenkernel/intern/cachefile.c

index 58770b3..74c0dca 100644 (file)
@@ -159,9 +159,6 @@ void BKE_cachefile_reload(const Main *bmain, CacheFile *cache_file)
 
 void BKE_cachefile_ensure_handle(const Main *bmain, CacheFile *cache_file)
 {
-       /* Assigning to a CoW copy is a bad idea; assign to the original instead. */
-       BLI_assert((cache_file->id.tag & LIB_TAG_COPIED_ON_WRITE) == 0);
-
        BLI_spin_lock(&spin);
        if (cache_file->handle_mutex == NULL) {
                cache_file->handle_mutex = BLI_mutex_alloc();
@@ -171,6 +168,8 @@ void BKE_cachefile_ensure_handle(const Main *bmain, CacheFile *cache_file)
        BLI_mutex_lock(cache_file->handle_mutex);
 
        if (cache_file->handle == NULL) {
+               /* Assigning to a CoW copy is a bad idea; assign to the original instead. */
+               BLI_assert((cache_file->id.tag & LIB_TAG_COPIED_ON_WRITE) == 0);
                BKE_cachefile_reload(bmain, cache_file);
        }