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:
parent
7a37db6125
commit
849f5bee19
|
@ -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;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue