[Bf-blender-cvs] [ca2756d] gooseberry: Updated cache reading function for the new duplilist structure.

[Lukas Tönne]

Commit: ca2756dbc325e36edde5bab79b8e45b08c471580
Author: Lukas Tönne
Date:   Sat Mar 14 14:41:08 2015 +0100
Branches: gooseberry
https://developer.blender.org/rBca2756dbc325e36edde5bab79b8e45b08c471580

Updated cache reading function for the new duplilist structure.

Now we store duplilists entirely inside a single alembic object for the
group. This allows using all the generated duplis, which would be
difficult if the alembic file had to define all the possible recursive
dupli relations that Blender allows, beside straightforward Group duplis.

===================================================================

M	source/blender/pointcache/alembic/abc_group.cpp

===================================================================

diff --git a/source/blender/pointcache/alembic/abc_group.cpp b/source/blender/pointcache/alembic/abc_group.cpp
index 8f7115f..65e807e 100644
--- a/source/blender/pointcache/alembic/abc_group.cpp
+++ b/source/blender/pointcache/alembic/abc_group.cpp
@@ -318,32 +318,23 @@ static void read_dupligroup_object(DupliGroupContext &ctx, IObject object, float
 	}
 }
 
-static void read_dupligroup_group(DupliGroupContext &ctx, IObject object, float frame);
-
-static void read_dupligroup_instance(DupliGroupContext &ctx, IObject instance, float frame)
+static void read_dupligroup_group(DupliGroupContext &ctx, IObject abc_group, float frame)
 {
-	if (instance.isInstanceRoot()) {
-		IObject object = IObject(instance.getPtr(), kWrapExisting);
-		if (object && GS(object.getName().c_str()) == ID_OB) {
-			DupliObjectData *data = ctx.find_dupli_data(object.getPtr());
-			if (data)
-				BKE_dupli_cache_add_instance(ctx.dupli_cache, ctx.get_transform(), data);
-			
-			IObject dup_group_object = object.getChild("dup_group");
-			if (dup_group_object)
-				read_dupligroup_group(ctx, dup_group_object, frame);
-		}
-	}
-}
-
-static void read_dupligroup_group(DupliGroupContext &ctx, IObject object, float frame)
-{
-	if (GS(object.getName().c_str()) == ID_GR) {
-		printf("reading group %s\n", object.getName().c_str());
-		size_t num_child = object.getNumChildren();
+	if (GS(abc_group.getName().c_str()) == ID_GR) {
+		size_t num_child = abc_group.getNumChildren();
 		
 		for (size_t i = 0; i < num_child; ++i) {
-			read_dupligroup_instance(ctx, object.getChild(i), frame);
+			IObject abc_dupli = abc_group.getChild(i);
+			
+			// TODO dupli offset, layers, etc.
+			
+			IObject abc_dupli_object = abc_dupli.getChild("object");
+			if (abc_dupli_object.isInstanceRoot()) {
+				DupliObjectData *dupli_data = ctx.find_dupli_data(abc_dupli_object.getPtr());
+				if (dupli_data) {
+					BKE_dupli_cache_add_instance(ctx.dupli_cache, ctx.get_transform(), dupli_data);
+				}
+			}
 		}
 	}
 }
@@ -359,18 +350,18 @@ PTCReadSampleResult abc_read_dupligroup(ReaderArchive *_archive, float frame, Gr
 	 */
 	ctx.build_object_map(G.main, dupgroup);
 	
-	IObject top = archive->archive.getTop();
-	size_t num_child = top.getNumChildren();
+	IObject abc_top = archive->archive.getTop();
+	IObject abc_group = archive->get_id_object((ID *)dupgroup);
+	if (!abc_group)
+		return PTC_READ_SAMPLE_INVALID;
 	
 	/* first create shared object data */
-	for (size_t i = 0; i < num_child; ++i) {
-		read_dupligroup_object(ctx, top.getChild(i), frame);
+	for (size_t i = 0; i < abc_top.getNumChildren(); ++i) {
+		read_dupligroup_object(ctx, abc_top.getChild(i), frame);
 	}
 	
 	/* now generate dupli instances for the dupgroup */
-	IObject dupgroup_object = top.getChild(dupgroup->id.name);
-	if (dupgroup_object)
-		read_dupligroup_group(ctx, dupgroup_object, frame);
+	read_dupligroup_group(ctx, abc_group, frame);
 	
 	return PTC_READ_SAMPLE_EXACT;
 }