Cleanup: ID makelocal code: remove useless and confusing debug code.

Thing is, user code should not have the responsibility to check that
libquery is valid. Such checks are only source of confusion and errors
as nobody will think about finding and updating them in some obscure
specific area of the code...
This commit is contained in:
Bastien Montagne 2020-05-25 15:38:36 +02:00
parent 7a37db6125
commit 849f5bee19
1 changed files with 0 additions and 14 deletions

View File

@ -1746,20 +1746,6 @@ static void library_make_local_copying_check(ID *id,
/* Our oh-so-beloved 'from' pointers... Those should always be ignored here, since the actual
* relation we want to check is in the other way around. */
if (entry->usage_flag & IDWALK_CB_LOOPBACK) {
#ifndef NDEBUG
/* Some debug checks to ensure we explicitly are aware of all 'loop-back' cases, since those
* may not always be manageable in the same way... */
switch (GS(par_id->name)) {
case ID_OB:
BLI_assert(((Object *)par_id)->proxy_from == (Object *)id);
break;
case ID_KE:
BLI_assert(((Key *)par_id)->from == id);
break;
default:
BLI_assert(0);
}
#endif
continue;
}