[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [35971] trunk/blender: quiet various warnings, also disable -Wdouble-promotion with cmake since it gives warnings with variable length args .

Campbell Barton ideasman42 at gmail.com
Sun Apr 3 12:04:16 CEST 2011


Revision: 35971
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=35971
Author:   campbellbarton
Date:     2011-04-03 10:04:16 +0000 (Sun, 03 Apr 2011)
Log Message:
-----------
quiet various warnings, also disable -Wdouble-promotion with cmake since it gives warnings with variable length args.

Modified Paths:
--------------
    trunk/blender/CMakeLists.txt
    trunk/blender/source/blender/blenkernel/intern/anim_sys.c
    trunk/blender/source/blender/blenlib/BLI_utildefines.h
    trunk/blender/source/blender/editors/interface/interface_icons.c
    trunk/blender/source/blender/editors/physics/physics_fluid.c
    trunk/blender/source/blender/editors/space_action/action_ops.c
    trunk/blender/source/blender/editors/space_graph/graph_ops.c
    trunk/blender/source/blender/editors/space_sequencer/sequencer_edit.c
    trunk/blender/source/blender/editors/space_sequencer/sequencer_scopes.c
    trunk/blender/source/blender/editors/space_view3d/view3d_view.c
    trunk/blender/source/blender/gpu/intern/gpu_draw.c
    trunk/blender/source/blender/makesrna/intern/rna_constraint.c
    trunk/blender/source/blender/makesrna/intern/rna_define.c
    trunk/blender/source/blender/makesrna/intern/rna_gpencil.c
    trunk/blender/source/blender/makesrna/intern/rna_wm_api.c
    trunk/blender/source/blender/modifiers/intern/MOD_explode.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_at.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_bricks.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_checker.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_compose.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_coord.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_curves.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_decompose.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_distance.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_hueSatVal.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_image.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_invert.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_math.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_mixRgb.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_output.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_proc.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_rotate.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_texture.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_translate.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_valToNor.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_valToRgb.c
    trunk/blender/source/blender/nodes/intern/TEX_nodes/TEX_viewer.c
    trunk/blender/source/blender/python/intern/bpy_rna.h
    trunk/blender/source/blender/python/intern/bpy_rna_anim.c

Modified: trunk/blender/CMakeLists.txt
===================================================================
--- trunk/blender/CMakeLists.txt	2011-04-03 07:48:33 UTC (rev 35970)
+++ trunk/blender/CMakeLists.txt	2011-04-03 10:04:16 UTC (rev 35971)
@@ -1059,7 +1059,8 @@
 	ADD_CHECK_C_COMPILER_FLAG(C_WARNINGS C_WARN_POINTER_ARITH -Wpointer-arith)
 	ADD_CHECK_C_COMPILER_FLAG(C_WARNINGS C_WARN_UNUSED_PARAMETER -Wunused-parameter)
 	ADD_CHECK_C_COMPILER_FLAG(C_WARNINGS C_WARN_WRITE_STRINGS -Wwrite-strings)
-	ADD_CHECK_C_COMPILER_FLAG(C_WARNINGS C_WARN_DOUBLE_PROMOTION -Wdouble-promotion -Wno-error=double-promotion)
+	# disable because it gives warnings for printf() & friends.
+	# ADD_CHECK_C_COMPILER_FLAG(C_WARNINGS C_WARN_DOUBLE_PROMOTION -Wdouble-promotion -Wno-error=double-promotion)
 	ADD_CHECK_C_COMPILER_FLAG(C_WARNINGS C_WARN_NO_ERROR_UNUSED_BUT_SET_VARIABLE -Wno-error=unused-but-set-variable)
 
 	ADD_CHECK_CXX_COMPILER_FLAG(CXX_WARNINGS CXX_WARN_ALL -Wall)

Modified: trunk/blender/source/blender/blenkernel/intern/anim_sys.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/anim_sys.c	2011-04-03 07:48:33 UTC (rev 35970)
+++ trunk/blender/source/blender/blenkernel/intern/anim_sys.c	2011-04-03 10:04:16 UTC (rev 35971)
@@ -1851,7 +1851,7 @@
  * ! This is exported so that keyframing code can use this for make use of it for anim layers support
  * > echannels: (list<NlaEvalChannels>) evaluation channels with calculated values
  */
-void animsys_evaluate_nla (ListBase *echannels, PointerRNA *ptr, AnimData *adt, float ctime) 
+static void animsys_evaluate_nla (ListBase *echannels, PointerRNA *ptr, AnimData *adt, float ctime)
 {
 	NlaTrack *nlt;
 	short track_index=0;

Modified: trunk/blender/source/blender/blenlib/BLI_utildefines.h
===================================================================
--- trunk/blender/source/blender/blenlib/BLI_utildefines.h	2011-04-03 07:48:33 UTC (rev 35970)
+++ trunk/blender/source/blender/blenlib/BLI_utildefines.h	2011-04-03 10:04:16 UTC (rev 35971)
@@ -206,7 +206,7 @@
 #  ifdef __GNUC__ /* just want to check if __func__ is available */
 #    define BLI_assert(a) \
 do { \
-	if (0 == (a)) { \
+	if (!(a)) { \
 		fprintf(stderr, \
 			"BLI_assert failed: %s, %s(), %d at \'%s\'\n", \
 			__FILE__, __func__, __LINE__, STRINGIFY(a)); \

Modified: trunk/blender/source/blender/editors/interface/interface_icons.c
===================================================================
--- trunk/blender/source/blender/editors/interface/interface_icons.c	2011-04-03 07:48:33 UTC (rev 35970)
+++ trunk/blender/source/blender/editors/interface/interface_icons.c	2011-04-03 10:04:16 UTC (rev 35971)
@@ -129,7 +129,7 @@
 /* ******************* STATIC LOCAL VARS ******************* */
 /* static here to cache results of icon directory scan, so it's not 
  * scanning the filesystem each time the menu is drawn */
-static struct ListBase iconfilelist = {0, 0};
+static struct ListBase iconfilelist = {NULL, NULL};
 static IconTexture icongltex = {0, 0, 0, 0.0f, 0.0f};
 
 /* **************************************************** */
@@ -144,7 +144,7 @@
 
 	new_icon = MEM_callocN(sizeof(Icon), "texicon");
 
-	new_icon->obj = 0; /* icon is not for library object */
+	new_icon->obj = NULL; /* icon is not for library object */
 	new_icon->type = 0;	
 
 	di = MEM_callocN(sizeof(DrawInfo), "drawinfo");
@@ -184,14 +184,14 @@
 
 	new_icon = MEM_callocN(sizeof(Icon), "texicon");
 
-	new_icon->obj = 0; /* icon is not for library object */
+	new_icon->obj = NULL; /* icon is not for library object */
 	new_icon->type = 0;
 
 	di = MEM_callocN(sizeof(DrawInfo), "drawinfo");
 	di->type= ICON_TYPE_VECTOR;
 	di->data.vector.func =drawFunc;
 
-	new_icon->drawinfo_free = 0;
+	new_icon->drawinfo_free = NULL;
 	new_icon->drawinfo = di;
 
 	BKE_icon_set(icon_id, new_icon);
@@ -667,7 +667,7 @@
 		if (dir[i].string) MEM_freeN(dir[i].string);
 	}
 	free(dir);
-	dir= 0;
+	dir= NULL;
 }
 
 static void free_iconfile_list(struct ListBase *list)

Modified: trunk/blender/source/blender/editors/physics/physics_fluid.c
===================================================================
--- trunk/blender/source/blender/editors/physics/physics_fluid.c	2011-04-03 07:48:33 UTC (rev 35970)
+++ trunk/blender/source/blender/editors/physics/physics_fluid.c	2011-04-03 10:04:16 UTC (rev 35971)
@@ -809,7 +809,7 @@
 	}
 }
 
-int fluidsimBake(bContext *C, ReportList *reports, Object *fsDomain)
+static int fluidsimBake(bContext *C, ReportList *reports, Object *fsDomain)
 {
 	Scene *scene= CTX_data_scene(C);
 	int i;

Modified: trunk/blender/source/blender/editors/space_action/action_ops.c
===================================================================
--- trunk/blender/source/blender/editors/space_action/action_ops.c	2011-04-03 07:48:33 UTC (rev 35970)
+++ trunk/blender/source/blender/editors/space_action/action_ops.c	2011-04-03 10:04:16 UTC (rev 35971)
@@ -39,8 +39,7 @@
 
 #include "BLI_blenlib.h"
 
-
-
+#include "ED_anim_api.h"
 #include "ED_transform.h"
 
 #include "action_intern.h"

Modified: trunk/blender/source/blender/editors/space_graph/graph_ops.c
===================================================================
--- trunk/blender/source/blender/editors/space_graph/graph_ops.c	2011-04-03 07:48:33 UTC (rev 35970)
+++ trunk/blender/source/blender/editors/space_graph/graph_ops.c	2011-04-03 10:04:16 UTC (rev 35971)
@@ -45,6 +45,7 @@
 
 #include "UI_view2d.h"
 
+#include "ED_anim_api.h"
 #include "ED_screen.h"
 #include "ED_transform.h"
 

Modified: trunk/blender/source/blender/editors/space_sequencer/sequencer_edit.c
===================================================================
--- trunk/blender/source/blender/editors/space_sequencer/sequencer_edit.c	2011-04-03 07:48:33 UTC (rev 35970)
+++ trunk/blender/source/blender/editors/space_sequencer/sequencer_edit.c	2011-04-03 10:04:16 UTC (rev 35971)
@@ -337,7 +337,7 @@
 		}
 		seq= seq->next;
 	}
-	return 0;
+	return NULL;
 }
 
 
@@ -461,12 +461,12 @@
 
 
 
-	if(last_seq==0 || last_seq->type!=SEQ_IMAGE) return;
+	if(last_seq==NULL || last_seq->type!=SEQ_IMAGE) return;
 	seqact= last_seq;	/* last_seq changes in alloc_sequence */
 
 	/* search sfile */
 //	sfile= scrarea_find_space_of_type(curarea, SPACE_FILE);
-	if(sfile==0) return;
+	if(sfile == NULL) return;
 
 	waitcursor(1);
 
@@ -479,7 +479,7 @@
 		seqact->len= seq->len;
 		calc_sequence(scene, seqact);
 
-		seq->strip= 0;
+		seq->strip= NULL;
 		seq_free_sequence(scene, seq);
 		BLI_remlink(ed->seqbasep, seq);
 
@@ -497,7 +497,7 @@
 	Scene *sce;
 	short event;
 
-	if(last_seq==0) return;
+	if(last_seq == NULL) return;
 
 	if(last_seq->type & SEQ_EFFECT) {
 		event = pupmenu("Change Effect%t"
@@ -590,7 +590,7 @@
 int seq_effect_find_selected(Scene *scene, Sequence *activeseq, int type, Sequence **selseq1, Sequence **selseq2, Sequence **selseq3, const char **error_str)
 {
 	Editing *ed = seq_give_editing(scene, FALSE);
-	Sequence *seq1= 0, *seq2= 0, *seq3= 0, *seq;
+	Sequence *seq1= NULL, *seq2= NULL, *seq3= NULL, *seq;
 	
 	*error_str= NULL;
 
@@ -604,9 +604,9 @@
 				return 0;
 			}
 			if((seq != activeseq) && (seq != seq2)) {
-								if(seq2==0) seq2= seq;
-								else if(seq1==0) seq1= seq;
-								else if(seq3==0) seq3= seq;
+								if(seq2 == NULL) seq2= seq;
+								else if(seq1 == NULL) seq1= seq;
+								else if(seq3 == NULL) seq3= seq;
 								else {
 									*error_str= "Can't apply effect to more than 3 sequence strips";
 									return 0;
@@ -617,7 +617,7 @@
        
 	/* make sequence selection a little bit more intuitive
 	   for 3 strips: the last-strip should be sequence3 */
-	if (seq3 != 0 && seq2 != 0) {
+	if (seq3 != NULL && seq2 != NULL) {
 		Sequence *tmp = seq2;
 		seq2 = seq3;
 		seq3 = tmp;
@@ -626,21 +626,21 @@
 
 	switch(get_sequence_effect_num_inputs(type)) {
 	case 0:
-		*selseq1 = *selseq2 = *selseq3 = 0;
+		*selseq1 = *selseq2 = *selseq3 = NULL;
 		return 1; /* succsess */
 	case 1:
-		if(seq2==0)  {
+		if(seq2==NULL)  {
 			*error_str= "Need at least one selected sequence strip";
 			return 0;
 		}
-		if(seq1==0) seq1= seq2;
-		if(seq3==0) seq3= seq2;
+		if(seq1==NULL) seq1= seq2;
+		if(seq3==NULL) seq3= seq2;
 	case 2:
-		if(seq1==0 || seq2==0) {
+		if(seq1==NULL || seq2==NULL) {
 			*error_str= "Need 2 selected sequence strips";
 			return 0;
 		}
-		if(seq3==0) seq3= seq2;
+		if(seq3 == NULL) seq3= seq2;
 	}
 	
 	if (seq1==NULL && seq2==NULL && seq3==NULL) {
@@ -722,7 +722,7 @@
 static Sequence *cut_seq_hard(Scene *scene, Sequence * seq, int cutframe)
 {
 	TransSeq ts;
-	Sequence *seqn = 0;
+	Sequence *seqn = NULL;
 	int skip_dup = FALSE;
 
 	/* backup values */
@@ -816,7 +816,7 @@
 static Sequence *cut_seq_soft(Scene *scene, Sequence * seq, int cutframe)
 {
 	TransSeq ts;
-	Sequence *seqn = 0;
+	Sequence *seqn = NULL;
 	int skip_dup = FALSE;
 
 	/* backup values */
@@ -2013,8 +2013,8 @@
 
 	BLI_movelisttolist(ed->seqbasep, &last_seq->seqbase);
 
-	last_seq->seqbase.first= 0;
-	last_seq->seqbase.last= 0;
+	last_seq->seqbase.first= NULL;
+	last_seq->seqbase.last= NULL;
 
 	BLI_remlink(ed->seqbasep, last_seq);
 	seq_free_sequence(scene, last_seq);

Modified: trunk/blender/source/blender/editors/space_sequencer/sequencer_scopes.c
===================================================================
--- trunk/blender/source/blender/editors/space_sequencer/sequencer_scopes.c	2011-04-03 07:48:33 UTC (rev 35970)
+++ trunk/blender/source/blender/editors/space_sequencer/sequencer_scopes.c	2011-04-03 10:04:16 UTC (rev 35971)
@@ -167,7 +167,7 @@
 	}
 
 	for (y = 0; y < ibuf->y; y++) {
-		unsigned char * last_p = 0;
+		unsigned char * last_p = NULL;
 
 		for (x = 0; x < ibuf->x; x++) {
 			unsigned char * rgb = src + 4 * (ibuf->x * y + x);
@@ -182,7 +182,7 @@
 			p += 4 * w;
 			scope_put_pixel(wtable, p);
 
-			if (last_p != 0) {
+			if (last_p != NULL) {
 				wform_put_line(w, last_p, p);
 			}
 			last_p = p;
@@ -213,7 +213,7 @@
 	}
 
 	for (y = 0; y < ibuf->y; y++) {
-		unsigned char * last_p = 0;
+		unsigned char * last_p = NULL;
 
 		for (x = 0; x < ibuf->x; x++) {
 			float * rgb = src + 4 * (ibuf->x * y + x);
@@ -231,7 +231,7 @@
 			p += 4 * w;
 			scope_put_pixel(wtable, p);
 
-			if (last_p != 0) {
+			if (last_p != NULL) {
 				wform_put_line(w, last_p, p);
 			}
 			last_p = p;
@@ -274,7 +274,7 @@
 	}
 
 	for (y = 0; y < ibuf->y; y++) {
-		unsigned char * last_p[3] = {0,0,0};
+		unsigned char *last_p[3] = {NULL, NULL, NULL};
 
 		for (x = 0; x < ibuf->x; x++) {
 			int c;

@@ Diff output truncated at 10240 characters. @@



More information about the Bf-blender-cvs mailing list