Revision: 51278
          
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=51278
Author:   campbellbarton
Date:     2012-10-11 23:46:12 +0000 (Thu, 11 Oct 2012)
Log Message:
-----------
fix for many RNA definitions having soft/hard ranges swapped, make this 
BLI_assert() on debug builds.

Modified Paths:
--------------
    trunk/blender/source/blender/editors/armature/poseobject.c
    trunk/blender/source/blender/editors/curve/editcurve.c
    trunk/blender/source/blender/editors/mesh/editmesh_select.c
    trunk/blender/source/blender/editors/mesh/editmesh_tools.c
    trunk/blender/source/blender/editors/render/render_shading.c
    trunk/blender/source/blender/editors/space_clip/clip_ops.c
    trunk/blender/source/blender/editors/space_clip/tracking_ops.c
    trunk/blender/source/blender/editors/space_image/image_ops.c
    trunk/blender/source/blender/editors/space_sequencer/sequencer_edit.c
    trunk/blender/source/blender/makesdna/DNA_windowmanager_types.h
    trunk/blender/source/blender/makesrna/intern/rna_define.c
    trunk/blender/source/blender/makesrna/intern/rna_image_api.c
    trunk/blender/source/blender/makesrna/intern/rna_main_api.c
    trunk/blender/source/blender/makesrna/intern/rna_texture_api.c
    trunk/blender/source/blender/windowmanager/WM_types.h
    trunk/blender/source/blender/windowmanager/intern/wm_operators.c

Modified: trunk/blender/source/blender/editors/armature/poseobject.c
===================================================================
--- trunk/blender/source/blender/editors/armature/poseobject.c  2012-10-11 
23:42:26 UTC (rev 51277)
+++ trunk/blender/source/blender/editors/armature/poseobject.c  2012-10-11 
23:46:12 UTC (rev 51278)
@@ -1511,7 +1511,7 @@
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
        
        /* properties */
-       RNA_def_int(ot->srna, "type", 0, 0, 10, "Bone Group Index", "", 0, 
INT_MAX);
+       RNA_def_int(ot->srna, "type", 0, 0, INT_MAX, "Bone Group Index", "", 0, 
10);
 }
 
 

Modified: trunk/blender/source/blender/editors/curve/editcurve.c
===================================================================
--- trunk/blender/source/blender/editors/curve/editcurve.c      2012-10-11 
23:42:26 UTC (rev 51277)
+++ trunk/blender/source/blender/editors/curve/editcurve.c      2012-10-11 
23:46:12 UTC (rev 51278)
@@ -4421,7 +4421,7 @@
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
        
        RNA_def_float_vector_xyz(ot->srna, "center", 3, NULL, -FLT_MAX, 
FLT_MAX, "Center", "Center in global view space", -FLT_MAX, FLT_MAX);
-       RNA_def_float_vector(ot->srna, "axis", 3, NULL, -1.0f, 1.0f, "Axis", 
"Axis in global view space", -FLT_MAX, FLT_MAX);
+       RNA_def_float_vector(ot->srna, "axis", 3, NULL, -FLT_MAX, FLT_MAX, 
"Axis", "Axis in global view space", -1.0f, 1.0f);
 }
 
 /***************** add vertex operator **********************/
@@ -5652,7 +5652,7 @@
        /* flags */
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 
-       RNA_def_int(ot->srna, "nth", 2, 2, 100, "Nth Selection", "", 1, 
INT_MAX);
+       RNA_def_int(ot->srna, "nth", 2, 2, INT_MAX, "Nth Selection", "", 2, 
100);
 }
 
 /********************** add duplicate operator *********************/

Modified: trunk/blender/source/blender/editors/mesh/editmesh_select.c
===================================================================
--- trunk/blender/source/blender/editors/mesh/editmesh_select.c 2012-10-11 
23:42:26 UTC (rev 51277)
+++ trunk/blender/source/blender/editors/mesh/editmesh_select.c 2012-10-11 
23:46:12 UTC (rev 51278)
@@ -2292,8 +2292,8 @@
        /* flags */
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 
-       RNA_def_int(ot->srna, "nth", 2, 2, 100, "Nth Selection", "", 1, 
INT_MAX);
-       RNA_def_int(ot->srna, "offset", 0, 0, 100, "Offset", "", 0, INT_MAX);
+       RNA_def_int(ot->srna, "nth", 2, 2, INT_MAX, "Nth Selection", "", 2, 
100);
+       RNA_def_int(ot->srna, "offset", 0, 0, INT_MAX, "Offset", "", 0, 100);
 }
 
 void em_setup_viewcontext(bContext *C, ViewContext *vc)

Modified: trunk/blender/source/blender/editors/mesh/editmesh_tools.c
===================================================================
--- trunk/blender/source/blender/editors/mesh/editmesh_tools.c  2012-10-11 
23:42:26 UTC (rev 51277)
+++ trunk/blender/source/blender/editors/mesh/editmesh_tools.c  2012-10-11 
23:46:12 UTC (rev 51278)
@@ -429,8 +429,8 @@
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
        
        /* props */
-       RNA_def_float(ot->srna, "offset", 2.0f, 0.0f, 100.0f, "Offset", "", 
0.0f, FLT_MAX);
-       RNA_def_int(ot->srna, "steps", 10, 0, 180, "Steps", "", 0, INT_MAX);
+       RNA_def_float(ot->srna, "offset", 2.0f, 0.0f, FLT_MAX, "Offset", "", 
0.0f, 100.0f);
+       RNA_def_int(ot->srna, "steps", 10, 0, INT_MAX, "Steps", "", 0, 180);
 }
 
 /* generic extern called extruder */
@@ -1607,7 +1607,7 @@
        /* flags */
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 
-       RNA_def_int(ot->srna, "repeat", 1, 1, 100, "Number of times to smooth 
the mesh", "", 1, INT_MAX);
+       RNA_def_int(ot->srna, "repeat", 1, 1, 1000, "Number of times to smooth 
the mesh", "", 1, 100);
        RNA_def_boolean(ot->srna, "xaxis", 1, "X-Axis", "Smooth along the X 
axis");
        RNA_def_boolean(ot->srna, "yaxis", 1, "Y-Axis", "Smooth along the Y 
axis");
        RNA_def_boolean(ot->srna, "zaxis", 1, "Z-Axis", "Smooth along the Z 
axis");
@@ -3542,7 +3542,7 @@
        RNA_def_float(ot->srna, "degrees", 90.0f, -FLT_MAX, FLT_MAX, "Degrees", 
"Degrees", -360.0f, 360.0f);
 
        RNA_def_float_vector(ot->srna, "center", 3, NULL, -FLT_MAX, FLT_MAX, 
"Center", "Center in global view space", -FLT_MAX, FLT_MAX);
-       RNA_def_float_vector(ot->srna, "axis", 3, NULL, -1.0f, 1.0f, "Axis", 
"Axis in global view space", -FLT_MAX, FLT_MAX);
+       RNA_def_float_vector(ot->srna, "axis", 3, NULL, -FLT_MAX, FLT_MAX, 
"Axis", "Axis in global view space", -1.0f, 1.0f);
 
 }
 
@@ -3666,8 +3666,8 @@
 
        RNA_def_float_vector(ot->srna, "center", 3, NULL, -FLT_MAX, FLT_MAX,
                             "Center", "Center in global view space", -FLT_MAX, 
FLT_MAX);
-       RNA_def_float_vector(ot->srna, "axis", 3, NULL, -1.0f, 1.0f,
-                            "Axis", "Axis in global view space", -FLT_MAX, 
FLT_MAX);
+       RNA_def_float_vector(ot->srna, "axis", 3, NULL, -FLT_MAX, FLT_MAX,
+                            "Axis", "Axis in global view space", -1.0f, 1.0f);
 }
 
 static int edbm_select_by_number_vertices_exec(bContext *C, wmOperator *op)

Modified: trunk/blender/source/blender/editors/render/render_shading.c
===================================================================
--- trunk/blender/source/blender/editors/render/render_shading.c        
2012-10-11 23:42:26 UTC (rev 51277)
+++ trunk/blender/source/blender/editors/render/render_shading.c        
2012-10-11 23:46:12 UTC (rev 51278)
@@ -747,7 +747,11 @@
        ot->flag = OPTYPE_REGISTER; /* no undo since this doesnt modify the 
env-map */
        
        /* properties */
-       prop = RNA_def_float_array(ot->srna, "layout", 12, 
default_envmap_layout, 0.0f, 0.0f, "File layout", "Flat array describing the 
X,Y position of each cube face in the output image, where 1 is the size of a 
face - order is [+Z -Z +Y -X -Y +X] (use -1 to skip a face)", 0.0f, 0.0f);
+       prop = RNA_def_float_array(ot->srna, "layout", 12, 
default_envmap_layout, 0.0f, 0.0f,
+                                  "File layout",
+                                  "Flat array describing the X,Y position of 
each cube face in the output image, "
+                                  "where 1 is the size of a face - order is 
[+Z -Z +Y -X -Y +X] "
+                                  "(use -1 to skip a face)", 0.0f, 0.0f);
        RNA_def_property_flag(prop, PROP_HIDDEN);
 
        WM_operator_properties_filesel(ot, FOLDERFILE | IMAGEFILE | MOVIEFILE, 
FILE_SPECIAL, FILE_SAVE,

Modified: trunk/blender/source/blender/editors/space_clip/clip_ops.c
===================================================================
--- trunk/blender/source/blender/editors/space_clip/clip_ops.c  2012-10-11 
23:42:26 UTC (rev 51277)
+++ trunk/blender/source/blender/editors/space_clip/clip_ops.c  2012-10-11 
23:46:12 UTC (rev 51278)
@@ -579,7 +579,7 @@
        ot->flag = OPTYPE_BLOCKING | OPTYPE_GRAB_POINTER;
 
        /* properties */
-       RNA_def_float(ot->srna, "factor", 0.0f, 0.0f, FLT_MAX,
+       RNA_def_float(ot->srna, "factor", 0.0f, -FLT_MAX, FLT_MAX,
                      "Factor", "Zoom factor, values higher than 1.0 zoom in, 
lower values zoom out", -FLT_MAX, FLT_MAX);
 }
 
@@ -700,7 +700,7 @@
        ot->poll = ED_space_clip_view_clip_poll;
 
        /* properties */
-       RNA_def_float(ot->srna, "ratio", 0.0f, 0.0f, FLT_MAX,
+       RNA_def_float(ot->srna, "ratio", 0.0f, -FLT_MAX, FLT_MAX,
                      "Ratio", "Zoom ratio, 1.0 is 1:1, higher is zoomed in, 
lower is zoomed out", -FLT_MAX, FLT_MAX);
 }
 

Modified: trunk/blender/source/blender/editors/space_clip/tracking_ops.c
===================================================================
--- trunk/blender/source/blender/editors/space_clip/tracking_ops.c      
2012-10-11 23:42:26 UTC (rev 51277)
+++ trunk/blender/source/blender/editors/space_clip/tracking_ops.c      
2012-10-11 23:46:12 UTC (rev 51278)
@@ -154,7 +154,7 @@
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 
        /* properties */
-       RNA_def_float_vector(ot->srna, "location", 2, NULL, -FLT_MIN, FLT_MAX,
+       RNA_def_float_vector(ot->srna, "location", 2, NULL, -FLT_MAX, FLT_MAX,
                             "Location", "Location of marker on frame", -1.0f, 
1.0f);
 }
 

Modified: trunk/blender/source/blender/editors/space_image/image_ops.c
===================================================================
--- trunk/blender/source/blender/editors/space_image/image_ops.c        
2012-10-11 23:42:26 UTC (rev 51277)
+++ trunk/blender/source/blender/editors/space_image/image_ops.c        
2012-10-11 23:46:12 UTC (rev 51278)
@@ -496,7 +496,7 @@
        ot->flag = OPTYPE_BLOCKING;
        
        /* properties */
-       RNA_def_float(ot->srna, "factor", 0.0f, 0.0f, FLT_MAX,
+       RNA_def_float(ot->srna, "factor", 0.0f, -FLT_MAX, FLT_MAX,
                      "Factor", "Zoom factor, values higher than 1.0 zoom in, 
lower values zoom out", -FLT_MAX, FLT_MAX);
 }
 
@@ -800,7 +800,7 @@
        ot->poll = space_image_main_area_poll;
        
        /* properties */
-       RNA_def_float(ot->srna, "ratio", 0.0f, 0.0f, FLT_MAX,
+       RNA_def_float(ot->srna, "ratio", 0.0f, -FLT_MAX, FLT_MAX,
                      "Ratio", "Zoom ratio, 1.0 is 1:1, higher is zoomed in, 
lower is zoomed out", -FLT_MAX, FLT_MAX);
 }
 

Modified: trunk/blender/source/blender/editors/space_sequencer/sequencer_edit.c
===================================================================
--- trunk/blender/source/blender/editors/space_sequencer/sequencer_edit.c       
2012-10-11 23:42:26 UTC (rev 51277)
+++ trunk/blender/source/blender/editors/space_sequencer/sequencer_edit.c       
2012-10-11 23:46:12 UTC (rev 51278)
@@ -1853,7 +1853,7 @@
        /* flags */
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 
-       RNA_def_int(ot->srna, "length", 1, 1, 1000, "Length", "Length of each 
frame", 1, INT_MAX);
+       RNA_def_int(ot->srna, "length", 1, 1, INT_MAX, "Length", "Length of 
each frame", 1, 1000);
 }
 
 
@@ -2188,7 +2188,7 @@
        ot->poll = ED_operator_sequencer_active;
 
        /* properties */
-       RNA_def_float(ot->srna, "ratio", 1.0f, 0.0f, FLT_MAX,
+       RNA_def_float(ot->srna, "ratio", 1.0f, -FLT_MAX, FLT_MAX,
                      "Ratio", "Zoom ratio, 1.0 is 1:1, higher is zoomed in, 
lower is zoomed out", -FLT_MAX, FLT_MAX);
 }
 

Modified: trunk/blender/source/blender/makesdna/DNA_windowmanager_types.h
===================================================================
--- trunk/blender/source/blender/makesdna/DNA_windowmanager_types.h     
2012-10-11 23:42:26 UTC (rev 51277)
+++ trunk/blender/source/blender/makesdna/DNA_windowmanager_types.h     
2012-10-11 23:46:12 UTC (rev 51278)
@@ -334,7 +334,7 @@
 #define OPERATOR_FLAGS_ALL             ((1<<5)-1)
 
 /* sanity checks for debug mode only */
-#define OPERATOR_RETVAL_CHECK(ret) BLI_assert(ret != 0 && (ret & 
OPERATOR_FLAGS_ALL) == ret)
+#define OPERATOR_RETVAL_CHECK(ret) (void)ret, BLI_assert(ret != 0 && (ret & 
OPERATOR_FLAGS_ALL) == ret)
 
 /* wmOperator flag */
 #define OP_GRAB_POINTER                        1


@@ Diff output truncated at 10240 characters. @@
_______________________________________________
Bf-blender-cvs mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to