Cleanup: use return args last and 'r_' prefix.
authorCampbell Barton <ideasman42@gmail.com>
Fri, 17 Mar 2017 22:39:36 +0000 (09:39 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 17 Mar 2017 22:39:36 +0000 (09:39 +1100)
source/blender/editors/interface/interface.c
source/blender/editors/util/numinput.c
source/blender/python/BPY_extern.h
source/blender/python/generic/py_capi_utils.c
source/blender/python/generic/py_capi_utils.h
source/blender/python/intern/bpy_interface.c

index 0328d09ef9a2bb456ff3bcf33f8898207cdb780c..57f3d228a17fcaf921d35db0b3c3e839f998917b 100644 (file)
@@ -2288,7 +2288,7 @@ char *ui_but_string_get_dynamic(uiBut *but, int *r_str_size)
 
 #ifdef WITH_PYTHON
 
-static bool ui_set_but_string_eval_num_unit(bContext *C, uiBut *but, const char *str, double *value)
+static bool ui_set_but_string_eval_num_unit(bContext *C, uiBut *but, const char *str, double *r_value)
 {
        char str_unit_convert[256];
        const int unit_type = UI_but_unit_type_get(but);
@@ -2300,13 +2300,13 @@ static bool ui_set_but_string_eval_num_unit(bContext *C, uiBut *but, const char
        bUnit_ReplaceString(str_unit_convert, sizeof(str_unit_convert), but->drawstr,
                            ui_get_but_scale_unit(but, 1.0), but->block->unit->system, RNA_SUBTYPE_UNIT_VALUE(unit_type));
 
-       return BPY_execute_string_as_number(C, str_unit_convert, value, true);
+       return BPY_execute_string_as_number(C, str_unit_convert, true, r_value);
 }
 
 #endif /* WITH_PYTHON */
 
 
-bool ui_but_string_set_eval_num(bContext *C, uiBut *but, const char *str, double *value)
+bool ui_but_string_set_eval_num(bContext *C, uiBut *but, const char *str, double *r_value)
 {
        bool ok = false;
 
@@ -2315,13 +2315,13 @@ bool ui_but_string_set_eval_num(bContext *C, uiBut *but, const char *str, double
        if (str[0] != '\0') {
                bool is_unit_but = (ui_but_is_float(but) && ui_but_is_unit(but));
                /* only enable verbose if we won't run again with units */
-               if (BPY_execute_string_as_number(C, str, value, is_unit_but == false)) {
+               if (BPY_execute_string_as_number(C, str, is_unit_but == false, r_value)) {
                        /* if the value parsed ok without unit conversion this button may still need a unit multiplier */
                        if (is_unit_but) {
                                char str_new[128];
 
-                               BLI_snprintf(str_new, sizeof(str_new), "%f", *value);
-                               ok = ui_set_but_string_eval_num_unit(C, but, str_new, value);
+                               BLI_snprintf(str_new, sizeof(str_new), "%f", *r_value);
+                               ok = ui_set_but_string_eval_num_unit(C, but, str_new, r_value);
                        }
                        else {
                                ok = true; /* parse normal string via py (no unit conversion needed) */
@@ -2329,7 +2329,7 @@ bool ui_but_string_set_eval_num(bContext *C, uiBut *but, const char *str, double
                }
                else if (is_unit_but) {
                        /* parse failed, this is a unit but so run replacements and parse again */
-                       ok = ui_set_but_string_eval_num_unit(C, but, str, value);
+                       ok = ui_set_but_string_eval_num_unit(C, but, str, r_value);
                }
        }
 
index e07831358d6802f11bea1d7e9e5f712f75004399..3727be1842ccc1582ee1ab96ed1d66a9be843a80 100644 (file)
@@ -497,7 +497,7 @@ bool handleNumInput(bContext *C, NumInput *n, const wmEvent *event)
                                    n->unit_sys, n->unit_type[idx]);
 
                /* Note: with angles, we always get values as radians here... */
-               if (BPY_execute_string_as_number(C, str_unit_convert, &val, false)) {
+               if (BPY_execute_string_as_number(C, str_unit_convert, false, &val)) {
                        n->val[idx] = (float)val;
                        n->val_flag[idx] &= ~NUM_INVALID;
                }
index 3148dab3c50a45cabec09f9b9b22880edcb83146..451ac271cdcf07997f102342d26670e548faec7e 100644 (file)
@@ -75,7 +75,7 @@ void BPY_thread_restore(BPy_ThreadStatePtr tstate);
 
 bool   BPY_execute_filepath(struct bContext *C, const char *filepath, struct ReportList *reports);
 bool   BPY_execute_text(struct bContext *C, struct Text *text, struct ReportList *reports, const bool do_jump);
-bool   BPY_execute_string_as_number(struct bContext *C, const char *expr, double *value, const bool verbose);
+bool   BPY_execute_string_as_number(struct bContext *C, const char *expr, const bool verbose, double *r_value);
 bool   BPY_execute_string_ex(struct bContext *C, const char *expr, bool use_eval);
 bool   BPY_execute_string(struct bContext *C, const char *expr);
 
index 7b2d58a126897e259900f7b6c819275837ca0a92..a2dc27f18e2b7f521e599fcb1ef4e01951367e33 100644 (file)
@@ -918,11 +918,11 @@ char *PyC_FlagSet_AsString(PyC_FlagSet *item)
        return cstring;
 }
 
-int PyC_FlagSet_ValueFromID_int(PyC_FlagSet *item, const char *identifier, int *value)
+int PyC_FlagSet_ValueFromID_int(PyC_FlagSet *item, const char *identifier, int *r_value)
 {
        for ( ; item->identifier; item++) {
                if (STREQ(item->identifier, identifier)) {
-                       *value = item->value;
+                       *r_value = item->value;
                        return 1;
                }
        }
@@ -930,9 +930,9 @@ int PyC_FlagSet_ValueFromID_int(PyC_FlagSet *item, const char *identifier, int *
        return 0;
 }
 
-int PyC_FlagSet_ValueFromID(PyC_FlagSet *item, const char *identifier, int *value, const char *error_prefix)
+int PyC_FlagSet_ValueFromID(PyC_FlagSet *item, const char *identifier, int *r_value, const char *error_prefix)
 {
-       if (PyC_FlagSet_ValueFromID_int(item, identifier, value) == 0) {
+       if (PyC_FlagSet_ValueFromID_int(item, identifier, r_value) == 0) {
                const char *enum_str = PyC_FlagSet_AsString(item);
                PyErr_Format(PyExc_ValueError,
                             "%s: '%.200s' not found in (%s)",
@@ -1006,7 +1006,7 @@ PyObject *PyC_FlagSet_FromBitfield(PyC_FlagSet *items, int flag)
  *
  * \note it is caller's responsibility to acquire & release GIL!
  */
-bool PyC_RunString_AsNumber(const char *expr, double *value, const char *filename)
+bool PyC_RunString_AsNumber(const char *expr, const char *filename, double *r_value)
 {
        PyObject *py_dict, *mod, *retval;
        bool ok = true;
@@ -1058,10 +1058,10 @@ bool PyC_RunString_AsNumber(const char *expr, double *value, const char *filenam
                        ok = false;
                }
                else if (!isfinite(val)) {
-                       *value = 0.0;
+                       *r_value = 0.0;
                }
                else {
-                       *value = val;
+                       *r_value = val;
                }
        }
 
index 04cfc8801ebb473e864b201132acb18bf8d5b98c..e0c66641e54de287a5bb2cc310fcc37e00fb4c9c 100644 (file)
@@ -75,12 +75,12 @@ typedef struct PyC_FlagSet {
 } PyC_FlagSet;
 
 char     *PyC_FlagSet_AsString(PyC_FlagSet *item);
-int       PyC_FlagSet_ValueFromID_int(PyC_FlagSet *item, const char *identifier, int *value);
-int       PyC_FlagSet_ValueFromID(PyC_FlagSet *item, const char *identifier, int *value, const char *error_prefix);
+int       PyC_FlagSet_ValueFromID_int(PyC_FlagSet *item, const char *identifier, int *r_value);
+int       PyC_FlagSet_ValueFromID(PyC_FlagSet *item, const char *identifier, int *r_value, const char *error_prefix);
 int       PyC_FlagSet_ToBitfield(PyC_FlagSet *items, PyObject *value, int *r_value, const char *error_prefix);
 PyObject *PyC_FlagSet_FromBitfield(PyC_FlagSet *items, int flag);
 
-bool PyC_RunString_AsNumber(const char *expr, double *value, const char *filename);
+bool PyC_RunString_AsNumber(const char *expr, const char *filename, double *r_value);
 
 int PyC_ParseBool(PyObject *o, void *p);
 
index 311f621e13b983f5140a20379607f4ec92f92091..1d561517eab303eac99dfebabe10f1feb81a07cf 100644 (file)
@@ -572,21 +572,23 @@ void BPY_DECREF_RNA_INVALIDATE(void *pyob_ptr)
 /**
  * \return success
  */
-bool BPY_execute_string_as_number(bContext *C, const char *expr, double *value, const bool verbose)
+bool BPY_execute_string_as_number(bContext *C, const char *expr, const bool verbose, double *r_value)
 {
        PyGILState_STATE gilstate;
        bool ok = true;
 
-       if (!value || !expr) return -1;
+       if (!r_value || !expr) {
+               return -1;
+       }
 
        if (expr[0] == '\0') {
-               *value = 0.0;
+               *r_value = 0.0;
                return ok;
        }
 
        bpy_context_set(C, &gilstate);
 
-       ok = PyC_RunString_AsNumber(expr, value, "<blender button>");
+       ok = PyC_RunString_AsNumber(expr, "<blender button>", r_value);
 
        if (ok == false) {
                if (verbose) {