[Bf-blender-cvs] [f21db78] multiview: Cleanup: minor edits

Campbell Barton noreply at git.blender.org
Tue Mar 17 13:56:10 CET 2015


Commit: f21db785d9daf90031155f9708b5ed6a2f289dc6
Author: Campbell Barton
Date:   Tue Mar 17 23:52:15 2015 +1100
Branches: multiview
https://developer.blender.org/rBf21db785d9daf90031155f9708b5ed6a2f289dc6

Cleanup: minor edits

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

M	source/blender/blenkernel/intern/image.c
M	source/blender/render/extern/include/RE_pipeline.h
M	source/blender/render/intern/source/envmap.c
M	source/blender/render/intern/source/pipeline.c
M	source/blender/render/intern/source/rendercore.c
M	source/blenderplayer/bad_level_call_stubs/stubs.c

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

diff --git a/source/blender/blenkernel/intern/image.c b/source/blender/blenkernel/intern/image.c
index ca8689d..ed5ad24 100644
--- a/source/blender/blenkernel/intern/image.c
+++ b/source/blender/blenkernel/intern/image.c
@@ -3301,10 +3301,11 @@ static ImBuf *image_load_movie_file(Image *ima, ImageUser *iuser, int frame)
 	return ibuf;
 }
 
-static ImBuf *load_image_single(Image *ima, ImageUser *iuser, int cfra,
-                                const size_t view_id,
-                                const bool has_packed,
-                                bool *r_assign)
+static ImBuf *load_image_single(
+        Image *ima, ImageUser *iuser, int cfra,
+        const size_t view_id,
+        const bool has_packed,
+        bool *r_assign)
 {
 	char filepath[FILE_MAX];
 	struct ImBuf *ibuf = NULL;
@@ -3319,8 +3320,8 @@ static ImBuf *load_image_single(Image *ima, ImageUser *iuser, int cfra,
 
 		imapf = BLI_findlink(&ima->packedfiles, view_id);
 		ibuf = IMB_ibImageFromMemory(
-			       (unsigned char *)imapf->packedfile->data, imapf->packedfile->size, flag,
-			       ima->colorspace_settings.name, "<packed data>");
+		       (unsigned char *)imapf->packedfile->data, imapf->packedfile->size, flag,
+		       ima->colorspace_settings.name, "<packed data>");
 	}
 	else {
 		ImageUser iuser_t;
diff --git a/source/blender/render/extern/include/RE_pipeline.h b/source/blender/render/extern/include/RE_pipeline.h
index d8f1fbb..b91beef6 100644
--- a/source/blender/render/extern/include/RE_pipeline.h
+++ b/source/blender/render/extern/include/RE_pipeline.h
@@ -314,7 +314,7 @@ void RE_zbuf_accumulate_vecblur(struct NodeBlurData *nbd, int xsize, int ysize,
 
 int RE_seq_render_active(struct Scene *scene, struct RenderData *rd);
 
-int RE_layers_have_name(struct RenderResult *result);
+bool RE_layers_have_name(struct RenderResult *result);
 
 /* shaded view or baking options */
 #define RE_BAKE_LIGHT				0	/* not listed in rna_scene.c -> can't be enabled! */
diff --git a/source/blender/render/intern/source/envmap.c b/source/blender/render/intern/source/envmap.c
index de1e255..33ca2b9 100644
--- a/source/blender/render/intern/source/envmap.c
+++ b/source/blender/render/intern/source/envmap.c
@@ -497,7 +497,7 @@ static void render_envmap(Render *re, EnvMap *env)
 			float *alpha;
 			float *rect;
 
-			rect= RE_RenderLayerGetPass(rl, SCE_PASS_COMBINED, re->viewname);
+			rect = RE_RenderLayerGetPass(rl, SCE_PASS_COMBINED, re->viewname);
 			ibuf = IMB_allocImBuf(envre->rectx, envre->recty, 24, IB_rect | IB_rectfloat);
 			memcpy(ibuf->rect_float, rect, ibuf->channels * ibuf->x * ibuf->y * sizeof(float));
 			
diff --git a/source/blender/render/intern/source/pipeline.c b/source/blender/render/intern/source/pipeline.c
index f039baa..6dbe748 100644
--- a/source/blender/render/intern/source/pipeline.c
+++ b/source/blender/render/intern/source/pipeline.c
@@ -317,9 +317,11 @@ Scene *RE_GetScene(Render *re)
 	return NULL;
 }
 
-/* Same as RE_AcquireResultImage but creating the necessary views to store the result
+/**
+ * Same as #RE_AcquireResultImage but creating the necessary views to store the result
  * fill provided result struct with a copy of thew views of what is done so far the
- * RenderResult.views ListBase needs to be freed after with RE_ReleaseResultViews */
+ * #RenderResult.views #ListBase needs to be freed after with #RE_ReleaseResultImageViews
+*/
 void RE_AcquireResultImageViews(Render *re, RenderResult *rr)
 {
 	memset(rr, 0, sizeof(RenderResult));
@@ -346,13 +348,15 @@ void RE_AcquireResultImageViews(Render *re, RenderResult *rr)
 
 			if (rl) {
 				if (rv->rectf == NULL) {
-					for (rview = (RenderView *)rr->views.first; rview; rview = rview->next)
+					for (rview = (RenderView *)rr->views.first; rview; rview = rview->next) {
 						rview->rectf = RE_RenderLayerGetPass(rl, SCE_PASS_COMBINED, rview->name);
+					}
 				}
 
 				if (rv->rectz == NULL) {
-					for (rview = (RenderView *)rr->views.first; rview; rview = rview->next)
+					for (rview = (RenderView *)rr->views.first; rview; rview = rview->next) {
 						rview->rectz = RE_RenderLayerGetPass(rl, SCE_PASS_Z, rview->name);
+					}
 				}
 			}
 
@@ -3419,7 +3423,7 @@ void RE_BlenderAnim(Render *re, Main *bmain, Scene *scene, Object *camera_overri
 		mh = BKE_movie_handle_get(scene->r.im_format.imtype);
 		re->movie_ctx_arr = MEM_mallocN(sizeof(void *) * totvideos, "Movies' Context");
 
-		for (i = 0; i < totvideos; i++){
+		for (i = 0; i < totvideos; i++) {
 			const char *suffix = BKE_scene_multiview_view_id_suffix_get(&re->r, i);
 
 			re->movie_ctx_arr[i] = mh->context_create();
@@ -3550,7 +3554,7 @@ void RE_BlenderAnim(Render *re, Main *bmain, Scene *scene, Object *camera_overri
 	/* end movie */
 	if (is_movie) {
 		size_t i;
-		for (i = 0; i < totvideos; i++){
+		for (i = 0; i < totvideos; i++) {
 			mh->end_movie(re->movie_ctx_arr[i]);
 			mh->context_free(re->movie_ctx_arr[i]);
 		}
@@ -3774,7 +3778,7 @@ bool RE_WriteEnvmapResult(struct ReportList *reports, Scene *scene, EnvMap *env,
 }
 
 /* used in the interface to decide whether to show layers */
-int RE_layers_have_name(struct RenderResult *rr)
+bool RE_layers_have_name(struct RenderResult *rr)
 {
 	switch (BLI_listbase_count_ex(&rr->layers, 2)) {
 		case 0:
diff --git a/source/blender/render/intern/source/rendercore.c b/source/blender/render/intern/source/rendercore.c
index 2461f55..ae13bc4 100644
--- a/source/blender/render/intern/source/rendercore.c
+++ b/source/blender/render/intern/source/rendercore.c
@@ -770,7 +770,7 @@ static void atm_tile(RenderPart *pa, RenderLayer *rl)
 			int sample;
 			
 			for (sample=0; sample<totsample; sample++) {
-				const float *zrect= RE_RenderLayerGetPass(rlpp[sample], SCE_PASS_Z, R.viewname) + od;
+				const float *zrect = RE_RenderLayerGetPass(rlpp[sample], SCE_PASS_Z, R.viewname) + od;
 				float *rgbrect = RE_RenderLayerGetPass(rlpp[sample], SCE_PASS_COMBINED, R.viewname) + 4*od;
 				float rgb[3] = {0};
 				bool done = false;
diff --git a/source/blenderplayer/bad_level_call_stubs/stubs.c b/source/blenderplayer/bad_level_call_stubs/stubs.c
index 4353228..6d768c7 100644
--- a/source/blenderplayer/bad_level_call_stubs/stubs.c
+++ b/source/blenderplayer/bad_level_call_stubs/stubs.c
@@ -229,7 +229,7 @@ void RE_exit_texture_rng() RET_NONE
 
 float *RE_RenderViewGetRectf(struct RenderResult *rr, int view_id) {STUB_ASSERT(0); return (float *) NULL;}
 float *RE_RenderViewGetRectz(struct RenderResult *rr, int view_id) {STUB_ASSERT(0); return (float *) NULL;}
-int RE_layers_have_name(struct RenderResult *result) {STUB_ASSERT(0); return 0;}
+bool RE_layers_have_name(struct RenderResult *result) {STUB_ASSERT(0); return 0;}
 void RE_engine_actview_set(struct RenderEngine *engine, const char *viewname) {STUB_ASSERT(0);}
 void RE_engine_get_camera_model_matrix(struct RenderEngine *engine, struct Object *camera, float *r_modelmat) {STUB_ASSERT(0);}
 float RE_engine_get_camera_shift_x(struct RenderEngine *engine, struct Object *camera) RET_ZERO




More information about the Bf-blender-cvs mailing list