[Bf-blender-cvs] [f2a1508] gooseberry: Set reader/writer archive as a separate step outside the constructors.

Lukas Tönne noreply at git.blender.org
Mon Mar 23 13:02:20 CET 2015


Commit: f2a1508a860f416bc468b5f7d1160e0e2cec313d
Author: Lukas Tönne
Date:   Tue Mar 3 16:01:09 2015 +0100
Branches: gooseberry
https://developer.blender.org/rBf2a1508a860f416bc468b5f7d1160e0e2cec313d

Set reader/writer archive as a separate step outside the constructors.

This allows nicer creation of readers/writers //before// actually
opening the archive. This in turns can simplify code quite a bit.

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

M	source/blender/blenkernel/BKE_cache_library.h
M	source/blender/blenkernel/intern/cache_library.c
M	source/blender/editors/io/io_cache_library.c
M	source/blender/pointcache/PTC_api.cpp
M	source/blender/pointcache/PTC_api.h
M	source/blender/pointcache/alembic/abc_cloth.cpp
M	source/blender/pointcache/alembic/abc_cloth.h
M	source/blender/pointcache/alembic/abc_mesh.cpp
M	source/blender/pointcache/alembic/abc_mesh.h
M	source/blender/pointcache/alembic/abc_particles.cpp
M	source/blender/pointcache/alembic/abc_particles.h
M	source/blender/pointcache/alembic/abc_reader.h
M	source/blender/pointcache/alembic/abc_writer.h
M	source/blender/pointcache/alembic/alembic.cpp
M	source/blender/pointcache/intern/ptc_types.h
M	source/blender/pointcache/intern/reader.cpp
M	source/blender/pointcache/intern/reader.h
M	source/blender/pointcache/intern/writer.cpp
M	source/blender/pointcache/intern/writer.h

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

diff --git a/source/blender/blenkernel/BKE_cache_library.h b/source/blender/blenkernel/BKE_cache_library.h
index aa14f82..a9b289d 100644
--- a/source/blender/blenkernel/BKE_cache_library.h
+++ b/source/blender/blenkernel/BKE_cache_library.h
@@ -96,7 +96,8 @@ void BKE_cache_library_group_update(struct Main *bmain, struct CacheLibrary *cac
 bool BKE_cache_archive_path_test(const char *path, ID *id, Library *lib);
 void BKE_cache_archive_path(const char *path, ID *id, Library *lib, char *result, int max);
 
-struct PTCWriterArchive *BKE_cache_library_writers(struct Scene *scene, int required_mode, struct CacheLibrary *cachelib, struct ListBase *writers);
+void BKE_cache_library_writers(struct Scene *scene, int required_mode, struct CacheLibrary *cachelib, struct ListBase *writers);
+struct PTCWriterArchive *BKE_cache_library_writers_open_archive(struct Scene *scene, struct CacheLibrary *cachelib, struct ListBase *writers);
 void BKE_cache_library_writers_free(struct PTCWriterArchive *archive, struct ListBase *writers);
 
 eCacheReadSampleResult BKE_cache_library_read_derived_mesh(struct Scene *scene, float frame, struct CacheLibrary *cachelib, struct Object *ob, struct DerivedMesh **r_dm);
diff --git a/source/blender/blenkernel/intern/cache_library.c b/source/blender/blenkernel/intern/cache_library.c
index 95c77f1..454bd21 100644
--- a/source/blender/blenkernel/intern/cache_library.c
+++ b/source/blender/blenkernel/intern/cache_library.c
@@ -654,15 +654,10 @@ static void cachelib_add_writer(ListBase *writers, struct PTCWriter *writer)
 	}
 }
 
-struct PTCWriterArchive *BKE_cache_library_writers(Scene *scene, int required_mode, CacheLibrary *cachelib, ListBase *writers)
+void BKE_cache_library_writers(Scene *scene, int required_mode, CacheLibrary *cachelib, ListBase *writers)
 {
-	char filename[FILE_MAX];
-	struct PTCWriterArchive *archive;
 	CacheItem *item;
 	
-	BKE_cache_archive_path(cachelib->filepath, (ID *)cachelib, cachelib->id.lib, filename, sizeof(filename));
-	archive = PTC_open_writer_archive(scene, filename);
-	
 	BLI_listbase_clear(writers);
 	
 	for (item = cachelib->items.first; item; item = item->next) {
@@ -677,30 +672,30 @@ struct PTCWriterArchive *BKE_cache_library_writers(Scene *scene, int required_mo
 			case CACHE_TYPE_DERIVED_MESH: {
 				CacheModifierData *cachemd = (CacheModifierData *)mesh_find_cache_modifier(scene, item->ob, required_mode);
 				if (cachemd)
-					cachelib_add_writer(writers, PTC_writer_cache_modifier(archive, name, item->ob, cachemd));
+					cachelib_add_writer(writers, PTC_writer_cache_modifier(name, item->ob, cachemd));
 				else
-					cachelib_add_writer(writers, PTC_writer_derived_final(archive, name, item->ob));
+					cachelib_add_writer(writers, PTC_writer_derived_final(name, item->ob));
 				break;
 			}
 			case CACHE_TYPE_HAIR: {
 				ParticleSystem *psys = (ParticleSystem *)BLI_findlink(&item->ob->particlesystem, item->index);
 				if (psys && psys->part && psys->part->type == PART_HAIR && psys->clmd) {
-					cachelib_add_writer(writers, PTC_writer_hair_dynamics(archive, name, item->ob, psys->clmd));
+					cachelib_add_writer(writers, PTC_writer_hair_dynamics(name, item->ob, psys->clmd));
 				}
 				break;
 			}
 			case CACHE_TYPE_HAIR_PATHS: {
 				ParticleSystem *psys = (ParticleSystem *)BLI_findlink(&item->ob->particlesystem, item->index);
 				if (psys && psys->part && psys->part->type == PART_HAIR) {
-					cachelib_add_writer(writers, PTC_writer_particles_pathcache_parents(archive, name, item->ob, psys));
-					cachelib_add_writer(writers, PTC_writer_particles_pathcache_children(archive, name, item->ob, psys));
+					cachelib_add_writer(writers, PTC_writer_particles_pathcache_parents(name, item->ob, psys));
+					cachelib_add_writer(writers, PTC_writer_particles_pathcache_children(name, item->ob, psys));
 				}
 				break;
 			}
 			case CACHE_TYPE_PARTICLES: {
 				ParticleSystem *psys = (ParticleSystem *)BLI_findlink(&item->ob->particlesystem, item->index);
 				if (psys && psys->part && psys->part->type != PART_HAIR) {
-					cachelib_add_writer(writers, PTC_writer_particles(archive, name, item->ob, psys));
+					cachelib_add_writer(writers, PTC_writer_particles(name, item->ob, psys));
 				}
 				break;
 			}
@@ -708,6 +703,21 @@ struct PTCWriterArchive *BKE_cache_library_writers(Scene *scene, int required_mo
 				break;
 		}
 	}
+}
+
+struct PTCWriterArchive *BKE_cache_library_writers_open_archive(Scene *scene, CacheLibrary *cachelib, ListBase *writers)
+{
+	char filename[FILE_MAX];
+	struct PTCWriterArchive *archive;
+	LinkData *link;
+	
+	BKE_cache_archive_path(cachelib->filepath, (ID *)cachelib, cachelib->id.lib, filename, sizeof(filename));
+	archive = PTC_open_writer_archive(scene, filename);
+	
+	for (link = writers->first; link; link = link->next) {
+		struct PTCWriter *writer = link->data;
+		PTC_writer_set_archive(writer, archive);
+	}
 	
 	return archive;
 }
@@ -726,7 +736,7 @@ void BKE_cache_library_writers_free(struct PTCWriterArchive *archive, ListBase *
 }
 
 
-static struct PTCReader *cache_library_reader_derived_mesh(CacheLibrary *cachelib, struct PTCReaderArchive *archive, Object *ob)
+static struct PTCReader *cache_library_reader_derived_mesh(CacheLibrary *cachelib, Object *ob)
 {
 	CacheItem *item;
 	
@@ -737,13 +747,13 @@ static struct PTCReader *cache_library_reader_derived_mesh(CacheLibrary *cacheli
 		char name[2*MAX_NAME];
 		BKE_cache_item_name(ob, CACHE_TYPE_DERIVED_MESH, -1, name);
 		
-		return PTC_reader_derived_mesh(archive, name, ob);
+		return PTC_reader_derived_mesh(name, ob);
 	}
 	
 	return NULL;
 }
 
-static struct PTCReader *cache_library_reader_hair_dynamics(CacheLibrary *cachelib, struct PTCReaderArchive *archive, Object *ob, ParticleSystem *psys)
+static struct PTCReader *cache_library_reader_hair_dynamics(CacheLibrary *cachelib, Object *ob, ParticleSystem *psys)
 {
 	CacheItem *item;
 	int index;
@@ -759,13 +769,13 @@ static struct PTCReader *cache_library_reader_hair_dynamics(CacheLibrary *cachel
 		char name[2*MAX_NAME];
 		BKE_cache_item_name(ob, CACHE_TYPE_HAIR, index, name);
 		
-		return PTC_reader_hair_dynamics(archive, name, ob, psys->clmd);
+		return PTC_reader_hair_dynamics(name, ob, psys->clmd);
 	}
 	
 	return NULL;
 }
 
-static struct PTCReader *cache_library_reader_particles(CacheLibrary *cachelib, struct PTCReaderArchive *archive, Object *ob, ParticleSystem *psys)
+static struct PTCReader *cache_library_reader_particles(CacheLibrary *cachelib, Object *ob, ParticleSystem *psys)
 {
 	CacheItem *item;
 	int index;
@@ -781,13 +791,13 @@ static struct PTCReader *cache_library_reader_particles(CacheLibrary *cachelib,
 		char name[2*MAX_NAME];
 		BKE_cache_item_name(ob, CACHE_TYPE_PARTICLES, index, name);
 		
-		return PTC_reader_particles(archive, name, ob, psys);
+		return PTC_reader_particles(name, ob, psys);
 	}
 	
 	return NULL;
 }
 
-static struct PTCReader *cache_library_reader_particles_pathcache_parents(CacheLibrary *cachelib, struct PTCReaderArchive *archive, Object *ob, ParticleSystem *psys)
+static struct PTCReader *cache_library_reader_particles_pathcache_parents(CacheLibrary *cachelib, Object *ob, ParticleSystem *psys)
 {
 	CacheItem *item;
 	int index;
@@ -803,13 +813,13 @@ static struct PTCReader *cache_library_reader_particles_pathcache_parents(CacheL
 		char name[2*MAX_NAME];
 		BKE_cache_item_name(ob, CACHE_TYPE_HAIR_PATHS, index, name);
 		
-		return PTC_reader_particles_pathcache_parents(archive, name, ob, psys);
+		return PTC_reader_particles_pathcache_parents(name, ob, psys);
 	}
 	
 	return NULL;
 }
 
-static struct PTCReader *cache_library_reader_particles_pathcache_children(CacheLibrary *cachelib, struct PTCReaderArchive *archive, Object *ob, ParticleSystem *psys)
+static struct PTCReader *cache_library_reader_particles_pathcache_children(CacheLibrary *cachelib, Object *ob, ParticleSystem *psys)
 {
 	CacheItem *item;
 	int index;
@@ -825,7 +835,7 @@ static struct PTCReader *cache_library_reader_particles_pathcache_children(Cache
 		char name[2*MAX_NAME];
 		BKE_cache_item_name(ob, CACHE_TYPE_HAIR_PATHS, index, name);
 		
-		return PTC_reader_particles_pathcache_children(archive, name, ob, psys);
+		return PTC_reader_particles_pathcache_children(name, ob, psys);
 	}
 	
 	return NULL;
@@ -834,110 +844,127 @@ static struct PTCReader *cache_library_reader_particles_pathcache_children(Cache
 
 eCacheReadSampleResult BKE_cache_library_read_derived_mesh(Scene *scene, float frame, CacheLibrary *cachelib, struct Object *ob, struct DerivedMesh **r_dm)
 {
-	char filename[FILE_MAX];
-	struct PTCReaderArchive *archive;
 	struct PTCReader *reader;
-	PTCReadSampleResult result;
+	eCacheReadSampleResult result = CACHE_READ_SAMPLE_INVALID;
 	
 	if (!(cachelib->flag & CACHE_LIBRARY_READ))
-		return PTC_READ_SAMPLE_INVALID;
-	
-	BKE_cache_archive_path(cachelib->filepath, (ID *)cachelib, cachelib->id.lib, filename, sizeof(filename));
-	archive = PTC_open_reader_archive(scene, filename);
+		return result;
 	
-	reader = cache_library_reader_derived_mesh(cachelib, archive, ob);
-	result = PTC_READ_SAMPLE_INVALID;
+	reader = cache_library_reader_derived_mesh(cachelib, ob);
 	if (reader) {
-		result = PTC_read_sample(reader, frame);
-		if (result != PTC_READ_SAMPLE_INVALID)
+		char filename[FILE_MAX];
+		struct PTCReaderArchive *archive;
+		
+		BKE_cache_archive_path(cachelib->filepath, (ID *)cachelib, cachelib->id.lib, filename, sizeof(filename));
+		archive = PTC_open_reader_archive(scene, filename);
+		PTC_reader_set_archive(reader, archive);
+		
+		result = BKE_cache_read_result(PTC_read_sample(reader, frame));
+		if (result != CACHE_READ_SAMPLE_INVALID)
 			*r_dm = PTC_reader_derived_mesh_acquire_result(reader);
+		
+		PTC_close_reader_archive(archive);
 	}
 	
-	PTC_close_reader_archive(archive);
-	
-	return BKE_cache_read_result(result);
+	return result;
 }
 
 eCacheReadSampleResult BKE_cache_library_read_hair_dynamics(Scene *scene, float frame, CacheLibrary *cachelib, Object *ob, ParticleSystem *psys)
 {
-	char filename[FILE_MAX];
-	struct PTCReaderArchive *archive;
 	struct PTCReader *reader;
-	PTCReadSampleResult result;
+	eCacheReadSampleResult result = CACHE_READ_SAMPLE_INVALID;
 	
 	if (!(cachelib->flag & CACHE_LIBRARY_REA

@@ Diff output truncated at 10240 characters. @@




More information about the Bf-blender-cvs mailing list