Replace of (id->lib != NULL) check by meaningful macro.

Idea is to replace hard-to-track (id->lib != NULL) 'is linked datablock' check everywhere in Blender
by a macro doing the same thing. This will allow to easily spot those checks in future, and more importantly,
to easily change it (see work done in asset-engine branch).

Note: did not touch to readfile.c, since there most of the time 'id->lib' check actually concerns the pointer,
and not a check whether ID is linked or not. Will have a closer look at it later.

Reviewers: campbellbarton, brecht, sergey

Differential Revision: https://developer.blender.org/D2082

1 files changed, 4 insertions, 4 deletions

diff --git a/source/blender/blenkernel/intern/lattice.c b/source/blender/blenkernel/intern/lattice.c
index 58c0a567116..07e6ba9062e 100644
--- a/source/blender/blenkernel/intern/lattice.c
+++ b/source/blender/blenkernel/intern/lattice.c
@@ -293,7 +293,7 @@ Lattice *BKE_lattice_copy(Lattice *lt)
 
 	ltn->editlatt = NULL;
 
-	if (lt->id.lib) {
+	if (ID_IS_LINKED_DATABLOCK(lt)) {
 		BKE_id_lib_local_paths(G.main, lt->id.lib, &ltn->id);
 	}
 
@@ -336,7 +336,7 @@ void BKE_lattice_make_local(Lattice *lt)
 	 * - mixed: make copy
 	 */
 	
-	if (lt->id.lib == NULL) return;
+	if (!ID_IS_LINKED_DATABLOCK(lt)) return;
 	if (lt->id.us == 1) {
 		id_clear_lib_data(bmain, &lt->id);
 		return;
@@ -344,7 +344,7 @@ void BKE_lattice_make_local(Lattice *lt)
 	
 	for (ob = bmain->object.first; ob && ELEM(false, is_lib, is_local); ob = ob->id.next) {
 		if (ob->data == lt) {
-			if (ob->id.lib) is_lib = true;
+			if (ID_IS_LINKED_DATABLOCK(ob)) is_lib = true;
 			else is_local = true;
 		}
 	}
@@ -361,7 +361,7 @@ void BKE_lattice_make_local(Lattice *lt)
 
 		for (ob = bmain->object.first; ob; ob = ob->id.next) {
 			if (ob->data == lt) {
-				if (ob->id.lib == NULL) {
+				if (!ID_IS_LINKED_DATABLOCK(ob)) {
 					ob->data = lt_new;
 					id_us_plus(&lt_new->id);
 					id_us_min(&lt->id);