Cleanup: Use new macro for deprecated ID types
Uses the macro introduced in b45f410b31
where it makes sense.
This commit is contained in:
parent
da1dd98101
commit
7b65086fdf
|
@ -311,7 +311,7 @@ void id_us_min(ID *id)
|
|||
const int limit = ID_FAKE_USERS(id);
|
||||
|
||||
if (id->us <= limit) {
|
||||
if (GS(id->name) != ID_IP) {
|
||||
if (!ID_TYPE_IS_DEPRECATED(GS(id->name))) {
|
||||
/* Do not assert on deprecated ID types, we cannot really ensure that their ID refcounting
|
||||
* is valid... */
|
||||
CLOG_ERROR(&LOG,
|
||||
|
@ -590,11 +590,9 @@ static int id_copy_libmanagement_cb(LibraryIDLinkCallbackData *cb_data)
|
|||
|
||||
bool BKE_id_copy_is_allowed(const ID *id)
|
||||
{
|
||||
#define LIB_ID_TYPES_NOCOPY \
|
||||
ID_LI, ID_SCR, ID_WM, ID_WS, /* Not supported */ \
|
||||
ID_IP /* Deprecated */
|
||||
#define LIB_ID_TYPES_NOCOPY ID_LI, ID_SCR, ID_WM, ID_WS /* Not supported */
|
||||
|
||||
return !ELEM(GS(id->name), LIB_ID_TYPES_NOCOPY);
|
||||
return !ID_TYPE_IS_DEPRECATED(GS(id->name)) && !ELEM(GS(id->name), LIB_ID_TYPES_NOCOPY);
|
||||
|
||||
#undef LIB_ID_TYPES_NOCOPY
|
||||
}
|
||||
|
|
|
@ -27,6 +27,11 @@ namespace blender::ed::outliner {
|
|||
|
||||
std::unique_ptr<TreeElementID> TreeElementID::createFromID(TreeElement &legacy_te, ID &id)
|
||||
{
|
||||
if (ID_TYPE_IS_DEPRECATED(GS(id.name))) {
|
||||
BLI_assert_msg(0, "Outliner trying to build tree-element for deprecated ID type");
|
||||
return nullptr;
|
||||
}
|
||||
|
||||
switch (ID_Type type = GS(id.name); type) {
|
||||
case ID_LI:
|
||||
return std::make_unique<TreeElementIDLibrary>(legacy_te, (Library &)id);
|
||||
|
@ -70,10 +75,9 @@ std::unique_ptr<TreeElementID> TreeElementID::createFromID(TreeElement &legacy_t
|
|||
case ID_PC:
|
||||
case ID_CF:
|
||||
return std::make_unique<TreeElementID>(legacy_te, id);
|
||||
/* Deprecated */
|
||||
case ID_IP:
|
||||
BLI_assert_msg(0, "Outliner trying to build tree-element for deprecated ID type");
|
||||
return nullptr;
|
||||
BLI_assert_unreachable();
|
||||
break;
|
||||
}
|
||||
|
||||
return nullptr;
|
||||
|
|
Loading…
Reference in New Issue