fix for many RNA definitions having soft/hard ranges swapped, make this BLI_assert...
authorCampbell Barton <ideasman42@gmail.com>
Thu, 11 Oct 2012 23:46:12 +0000 (23:46 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Thu, 11 Oct 2012 23:46:12 +0000 (23:46 +0000)
16 files changed:
source/blender/editors/armature/poseobject.c
source/blender/editors/curve/editcurve.c
source/blender/editors/mesh/editmesh_select.c
source/blender/editors/mesh/editmesh_tools.c
source/blender/editors/render/render_shading.c
source/blender/editors/space_clip/clip_ops.c
source/blender/editors/space_clip/tracking_ops.c
source/blender/editors/space_image/image_ops.c
source/blender/editors/space_sequencer/sequencer_edit.c
source/blender/makesdna/DNA_windowmanager_types.h
source/blender/makesrna/intern/rna_define.c
source/blender/makesrna/intern/rna_image_api.c
source/blender/makesrna/intern/rna_main_api.c
source/blender/makesrna/intern/rna_texture_api.c
source/blender/windowmanager/WM_types.h
source/blender/windowmanager/intern/wm_operators.c

index 189b2e977c21ca0b1796a96a5caae026f2ef4546..365500a517b2fe3ae7098b9c4d437e3cb203870a 100644 (file)
@@ -1511,7 +1511,7 @@ void POSE_OT_group_assign(wmOperatorType *ot)
        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);
 }
 
 
index 42b42a594ce6e30ad121500ff21f6f9151969c63..98dc8aa6eb9709b113901a4fcb554f5db9ea0ff6 100644 (file)
@@ -4421,7 +4421,7 @@ void CURVE_OT_spin(wmOperatorType *ot)
        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 @@ void CURVE_OT_select_nth(wmOperatorType *ot)
        /* 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 *********************/
index 32643cb20cd549e387e016cd521d76611e33522c..fe3860de839ed63ce3858ca3b540a64f2c48c76a 100644 (file)
@@ -2292,8 +2292,8 @@ void MESH_OT_select_nth(wmOperatorType *ot)
        /* 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)
index 891e148778143bf8e84e535f1dc14ef5bf86ec66..44810048ad2c007760f02451f9f287e3b51d3172 100644 (file)
@@ -429,8 +429,8 @@ void MESH_OT_extrude_repeat(wmOperatorType *ot)
        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 @@ void MESH_OT_vertices_smooth(wmOperatorType *ot)
        /* 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 @@ void MESH_OT_spin(wmOperatorType *ot)
        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 @@ void MESH_OT_screw(wmOperatorType *ot)
 
        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)
index 8bdd4e544e88b6a77ddb9bdd8ea7235842f80273..25a01d38dd575ecd0e8e9b022e26222077d6a3c9 100644 (file)
@@ -747,7 +747,11 @@ void TEXTURE_OT_envmap_save(wmOperatorType *ot)
        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,
index 5c338f3e6f16526de5ea02790a5cb7981cca760c..be5fa08f93151e0d4263f7447f7e1c7740ebe1e1 100644 (file)
@@ -579,7 +579,7 @@ void CLIP_OT_view_zoom(wmOperatorType *ot)
        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 @@ void CLIP_OT_view_zoom_ratio(wmOperatorType *ot)
        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);
 }
 
index 8dc28bbaee0f2379f50fbb645a9f0e21b98c7e46..1af6237689c8cdb66cd15f2a6dfd5bc2af003b87 100644 (file)
@@ -154,7 +154,7 @@ void CLIP_OT_add_marker(wmOperatorType *ot)
        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);
 }
 
index c7ad82dec85726a172610e4ab92630f51fcbb9c3..405fb5a9243d9088fd54bd21fdc0a40716a57e30 100644 (file)
@@ -496,7 +496,7 @@ void IMAGE_OT_view_zoom(wmOperatorType *ot)
        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 @@ void IMAGE_OT_view_zoom_ratio(wmOperatorType *ot)
        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);
 }
 
index 459a8d54d129f3c62a6ee3835fd38dd0e415e1f9..95d827f9392826e6319f5065f9fe21f1fef5efe7 100644 (file)
@@ -1853,7 +1853,7 @@ void SEQUENCER_OT_images_separate(wmOperatorType *ot)
        /* 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 @@ void SEQUENCER_OT_view_zoom_ratio(wmOperatorType *ot)
        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);
 }
 
index 780ca0b5878ff2e7be8997c2465d67e08bb90db8..2294abc0735951882037019b19c582f73581fcbd 100644 (file)
@@ -334,7 +334,7 @@ typedef struct wmOperator {
 #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
index 45092d09ce1a416febe598774fa7cf3972bdd73f..d4e0ba68d9d7a846870a6566606c40f9d634c4dd 100644 (file)
 
 #include "rna_internal.h"
 
+
+#ifdef DEBUG
+#  define ASSERT_SOFT_HARD_LIMITS \
+       if (softmin < hardmin || softmax > hardmax) { \
+               fprintf(stderr, "Error with soft/hard limits: %s.%s\n", CONTAINER_RNA_ID(cont), identifier); \
+               BLI_assert(!"invalid soft/hard limits"); \
+       } (void)0
+#else
+#  define  ASSERT_SOFT_HARD_LIMITS (void)0
+#endif
+
 /* Global used during defining */
 
 BlenderDefRNA DefRNA = {NULL, {NULL, NULL}, {NULL, NULL}, NULL, 0, 0, 0, 1};
@@ -2248,12 +2259,15 @@ PropertyRNA *RNA_def_boolean_vector(StructOrFunctionRNA *cont_, const char *iden
        return prop;
 }
 
-PropertyRNA *RNA_def_int(StructOrFunctionRNA *cont_, const char *identifier, int default_value, int hardmin,
-                         int hardmax, const char *ui_name, const char *ui_description, int softmin, int softmax)
+PropertyRNA *RNA_def_int(StructOrFunctionRNA *cont_, const char *identifier, int default_value,
+                         int hardmin, int hardmax, const char *ui_name, const char *ui_description,
+                         int softmin, int softmax)
 {
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
        
+       ASSERT_SOFT_HARD_LIMITS;
+
        prop = RNA_def_property(cont, identifier, PROP_INT, PROP_NONE);
        RNA_def_property_int_default(prop, default_value);
        if (hardmin != hardmax) RNA_def_property_range(prop, hardmin, hardmax);
@@ -2270,6 +2284,8 @@ PropertyRNA *RNA_def_int_vector(StructOrFunctionRNA *cont_, const char *identifi
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
        
+       ASSERT_SOFT_HARD_LIMITS;
+
        prop = RNA_def_property(cont, identifier, PROP_INT, PROP_XYZ); /* XXX */
        if (len != 0) RNA_def_property_array(prop, len);
        if (default_value) RNA_def_property_int_array_default(prop, default_value);
@@ -2287,6 +2303,8 @@ PropertyRNA *RNA_def_int_array(StructOrFunctionRNA *cont_, const char *identifie
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
        
+       ASSERT_SOFT_HARD_LIMITS;
+
        prop = RNA_def_property(cont, identifier, PROP_INT, PROP_NONE);
        if (len != 0) RNA_def_property_array(prop, len);
        if (default_value) RNA_def_property_int_array_default(prop, default_value);
@@ -2426,6 +2444,8 @@ PropertyRNA *RNA_def_float(StructOrFunctionRNA *cont_, const char *identifier, f
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
        
+       ASSERT_SOFT_HARD_LIMITS;
+
        prop = RNA_def_property(cont, identifier, PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_default(prop, default_value);
        if (hardmin != hardmax) RNA_def_property_range(prop, hardmin, hardmax);
@@ -2442,6 +2462,8 @@ PropertyRNA *RNA_def_float_vector(StructOrFunctionRNA *cont_, const char *identi
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
        
+       ASSERT_SOFT_HARD_LIMITS;
+
        prop = RNA_def_property(cont, identifier, PROP_FLOAT, PROP_XYZ);
        if (len != 0) RNA_def_property_array(prop, len);
        if (default_value) RNA_def_property_float_array_default(prop, default_value);
@@ -2472,6 +2494,8 @@ PropertyRNA *RNA_def_float_color(StructOrFunctionRNA *cont_, const char *identif
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
        
+       ASSERT_SOFT_HARD_LIMITS;
+
        prop = RNA_def_property(cont, identifier, PROP_FLOAT, PROP_COLOR);
        if (len != 0) RNA_def_property_array(prop, len);
        if (default_value) RNA_def_property_float_array_default(prop, default_value);
@@ -2489,10 +2513,9 @@ PropertyRNA *RNA_def_float_matrix(StructOrFunctionRNA *cont_, const char *identi
 {
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
-       int length[2];
+       const int length[2] = {rows, columns};
 
-       length[0] = rows;
-       length[1] = columns;
+       ASSERT_SOFT_HARD_LIMITS;
 
        prop = RNA_def_property(cont, identifier, PROP_FLOAT, PROP_MATRIX);
        RNA_def_property_multi_array(prop, 2, length);
@@ -2510,7 +2533,9 @@ PropertyRNA *RNA_def_float_rotation(StructOrFunctionRNA *cont_, const char *iden
 {
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
-       
+
+       ASSERT_SOFT_HARD_LIMITS;
+
        prop = RNA_def_property(cont, identifier, PROP_FLOAT, (len != 0) ? PROP_EULER : PROP_ANGLE);
        if (len != 0) {
                RNA_def_property_array(prop, len);
@@ -2534,6 +2559,8 @@ PropertyRNA *RNA_def_float_array(StructOrFunctionRNA *cont_, const char *identif
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
        
+       ASSERT_SOFT_HARD_LIMITS;
+
        prop = RNA_def_property(cont, identifier, PROP_FLOAT, PROP_NONE);
        if (len != 0) RNA_def_property_array(prop, len);
        if (default_value) RNA_def_property_float_array_default(prop, default_value);
@@ -2551,6 +2578,8 @@ PropertyRNA *RNA_def_float_percentage(StructOrFunctionRNA *cont_, const char *id
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
        
+       ASSERT_SOFT_HARD_LIMITS;
+
        prop = RNA_def_property(cont, identifier, PROP_FLOAT, PROP_PERCENTAGE);
        RNA_def_property_float_default(prop, default_value);
        if (hardmin != hardmax) RNA_def_property_range(prop, hardmin, hardmax);
@@ -2567,6 +2596,8 @@ PropertyRNA *RNA_def_float_factor(StructOrFunctionRNA *cont_, const char *identi
        ContainerRNA *cont = cont_;
        PropertyRNA *prop;
        
+       ASSERT_SOFT_HARD_LIMITS;
+
        prop = RNA_def_property(cont, identifier, PROP_FLOAT, PROP_FACTOR);
        RNA_def_property_float_default(prop, default_value);
        if (hardmin != hardmax) RNA_def_property_range(prop, hardmin, hardmax);
index 7766dff427367796058aa19a43d733b0dbcbe102..ab50190905245675b94360201a8104883480e08a 100644 (file)
@@ -297,9 +297,9 @@ void RNA_api_image(StructRNA *srna)
        func = RNA_def_function(srna, "scale", "rna_Image_scale");
        RNA_def_function_ui_description(func, "Scale the image in pixels");
        RNA_def_function_flag(func, FUNC_USE_REPORTS);
-       parm = RNA_def_int(func, "width", 0, 1, 10000, "", "Width", 1, 10000);
+       parm = RNA_def_int(func, "width", 1, 1, 10000, "", "Width", 1, 10000);
        RNA_def_property_flag(parm, PROP_REQUIRED);
-       parm = RNA_def_int(func, "height", 0, 1, 10000, "", "Height", 1, 10000);
+       parm = RNA_def_int(func, "height", 1, 1, 10000, "", "Height", 1, 10000);
        RNA_def_property_flag(parm, PROP_REQUIRED);
 
        func = RNA_def_function(srna, "gl_touch", "rna_Image_gl_touch");
index cd9b3965713d4111c23761b3ed63cabc5319a5f6..8e21ab3f7e93228d0bc7427457d9a591218b5d06 100644 (file)
@@ -990,9 +990,9 @@ void RNA_def_main_images(BlenderRNA *brna, PropertyRNA *cprop)
        RNA_def_function_ui_description(func, "Add a new image to the main database");
        parm = RNA_def_string(func, "name", "Image", 0, "", "New name for the datablock");
        RNA_def_property_flag(parm, PROP_REQUIRED);
-       parm = RNA_def_int(func, "width", 1024, 1, INT_MAX, "", "Width of the image", 0, INT_MAX);
+       parm = RNA_def_int(func, "width", 1024, 1, INT_MAX, "", "Width of the image", 1, INT_MAX);
        RNA_def_property_flag(parm, PROP_REQUIRED);
-       parm = RNA_def_int(func, "height", 1024, 1, INT_MAX, "", "Height of the image", 0, INT_MAX);
+       parm = RNA_def_int(func, "height", 1024, 1, INT_MAX, "", "Height of the image", 1, INT_MAX);
        RNA_def_property_flag(parm, PROP_REQUIRED);
        RNA_def_boolean(func, "alpha", 0, "Alpha", "Use alpha channel");
        RNA_def_boolean(func, "float_buffer", 0, "Float Buffer", "Create an image with floating point color");
index a288051095849736aff20c3071dcf7b5a86aaa1f..5be9d3a0dec00abc4bba4345fab7e391b109cbd4 100644 (file)
@@ -120,10 +120,10 @@ void RNA_api_environment_map(StructRNA *srna)
 
        RNA_def_pointer(func, "scene", "Scene", "", "Overrides the scene from which image parameters are taken");
 
-       RNA_def_float_array(func, "layout", 12, default_layout, 0.0f, 0.0f, "File layout",
+       RNA_def_float_array(func, "layout", 12, default_layout, 0.0f, 1000.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);
+                           "(use -1 to skip a face)", 0.0f, 1000.0f);
 }
 
 #endif
index ad828cef6d5386f1d8cbcfe435c8f33d50df9987..764d274bfba773faae6ce18947ecde8c0064f7cc 100644 (file)
@@ -539,7 +539,11 @@ typedef struct wmOperatorType {
 
        /* verify if the operator can be executed in the current context, note
         * that the operator might still fail to execute even if this return true */
-       int (*poll)(struct bContext *);
+       int (*poll)(struct bContext *)
+#ifdef __GNUC__
+       __attribute__((warn_unused_result))
+#endif
+       ;
 
        /* optional panel for redo and repeat, autogenerated if not set */
        void (*ui)(struct bContext *, struct wmOperator *);
@@ -563,7 +567,11 @@ typedef struct wmOperatorType {
        /* only used for operators defined with python
         * use to store pointers to python functions */
        void *pyop_data;
-       int (*pyop_poll)(struct bContext *, struct wmOperatorType *ot);
+       int (*pyop_poll)(struct bContext *, struct wmOperatorType *ot)
+#ifdef __GNUC__
+       __attribute__((warn_unused_result))
+#endif
+       ;
 
        /* RNA integration */
        ExtensionRNA ext;
index ec25e8dcffb3c45948cca0974b13740f243a245b..19c2a58fdfc6b65f5d6ea1e17d95c8c36d12ff30 100644 (file)
@@ -1233,7 +1233,7 @@ static void WM_OT_debug_menu(wmOperatorType *ot)
        ot->exec = wm_debug_menu_exec;
        ot->poll = WM_operator_winactive;
        
-       RNA_def_int(ot->srna, "debug_value", 0, -10000, 10000, "Debug Value", "", INT_MIN, INT_MAX);
+       RNA_def_int(ot->srna, "debug_value", 0, SHRT_MIN, SHRT_MAX, "Debug Value", "", -10000, 10000);
 }