diff options
| author | Bastien Montagne <bastien@blender.org> | 2022-05-17 16:08:18 +0300 |
|---|---|---|
| committer | Bastien Montagne <bastien@blender.org> | 2022-05-17 17:06:54 +0300 |
| commit | ff2d6c2ba8e8f667a0aac7a79c77b58d87f45a73 (patch) | |
| tree | 6c6b54bfa9cf367867e347ee0ac036dc85e475d8 /source/blender/blenkernel/intern/lib_remap.c | |
| parent | c8b740cc0012c7414ac12b569556b0b309ada662 (diff) | |
Cleanup: Use `switch` and `BLI_assert_unreachable()` more.
Replace some `if/else if` chains by proper `switch` statement.
Replace some `BLI_assert(0)` calls by `BLI_assert_unreachable()` ones.
Diffstat (limited to 'source/blender/blenkernel/intern/lib_remap.c')
| -rw-r--r-- | source/blender/blenkernel/intern/lib_remap.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/source/blender/blenkernel/intern/lib_remap.c b/source/blender/blenkernel/intern/lib_remap.c index 971db852463..583eb3f03ca 100644 --- a/source/blender/blenkernel/intern/lib_remap.c +++ b/source/blender/blenkernel/intern/lib_remap.c @@ -75,6 +75,7 @@ static void foreach_libblock_remap_callback_skip(const ID *UNUSED(id_owner), { ID *id = *id_ptr; BLI_assert(id != NULL); + if (is_indirect) { id->runtime.remap.skipped_indirect++; } @@ -82,8 +83,9 @@ static void foreach_libblock_remap_callback_skip(const ID *UNUSED(id_owner), id->runtime.remap.skipped_direct++; } else { - BLI_assert(0); + BLI_assert_unreachable(); } + if (cb_flag & IDWALK_CB_USER) { id->runtime.remap.skipped_refcounted++; } |