Code cleanup: use 'const' for arrays (python)

diff --git a/source/blender/python/bmesh/bmesh_py_ops_call.c b/source/blender/python/bmesh/bmesh_py_ops_call.c
index f4fe1ede088a29292abc257bd412e51fb6fbea7e..9c9b69186abf34001e36b876e72ab714f6c8257e 100644 (file)
--- a/source/blender/python/bmesh/bmesh_py_ops_call.c
+++ b/source/blender/python/bmesh/bmesh_py_ops_call.c
@@ -768,7 +768,7 @@ PyObject *BPy_BMO_call(BPy_BMeshOpFunc *self, PyObject *args, PyObject *kw)
                        /* temp code, strip off '.out' while we keep this convention */
                        {
                                char slot_name_strip[MAX_SLOTNAME];
-                               char *ch = strchr(slot->slot_name, '.');  /* can't fail! */
+                               const char *ch = strchr(slot->slot_name, '.');  /* can't fail! */
                                int tot = ch - slot->slot_name;
                                BLI_assert(ch != NULL);
                                memcpy(slot_name_strip, slot->slot_name, tot);

diff --git a/source/blender/python/generic/bgl.c b/source/blender/python/generic/bgl.c
index d1d0ec96315e98c714d8a11047aafff38e86ac0e..292d3317a2eddba3a824592b4449c3f76a121eee 100644 (file)
--- a/source/blender/python/generic/bgl.c
+++ b/source/blender/python/generic/bgl.c
@@ -1905,7 +1905,7 @@ PyObject *BPyInit_bgl(void)
 static PyObject *Method_ShaderSource(PyObject *UNUSED(self), PyObject *args)
 {
        unsigned int shader;
-       char *source;
+       const char *source;
 
        if (!PyArg_ParseTuple(args, "Is", &shader, &source))
                return NULL;

diff --git a/source/blender/python/generic/blf_py_api.c b/source/blender/python/generic/blf_py_api.c
index 768c5506b8c319177124496576aa67f0974c3849..655542e320aeb5a663cb5f4fdd0e893f96f68f9d 100644 (file)
--- a/source/blender/python/generic/blf_py_api.c
+++ b/source/blender/python/generic/blf_py_api.c
@@ -146,7 +146,7 @@ PyDoc_STRVAR(py_blf_draw_doc,
 );
 static PyObject *py_blf_draw(PyObject *UNUSED(self), PyObject *args)
 {
-       char *text;
+       const char *text;
        int text_length;
        int fontid;
 
@@ -172,7 +172,7 @@ PyDoc_STRVAR(py_blf_dimensions_doc,
 );
 static PyObject *py_blf_dimensions(PyObject *UNUSED(self), PyObject *args)
 {
-       char *text;
+       const char *text;
        float r_width, r_height;
        PyObject *ret;
        int fontid;
@@ -356,7 +356,7 @@ PyDoc_STRVAR(py_blf_load_doc,
 );
 static PyObject *py_blf_load(PyObject *UNUSED(self), PyObject *args)
 {
-       char *filename;
+       const char *filename;
 
        if (!PyArg_ParseTuple(args, "s:blf.load", &filename))
                return NULL;
@@ -374,7 +374,7 @@ PyDoc_STRVAR(py_blf_unload_doc,
 );
 static PyObject *py_blf_unload(PyObject *UNUSED(self), PyObject *args)
 {
-       char *filename;
+       const char *filename;
 
        if (!PyArg_ParseTuple(args, "s:blf.unload", &filename))
                return NULL;

diff --git a/source/blender/python/generic/bpy_internal_import.c b/source/blender/python/generic/bpy_internal_import.c
index af41c3b426bda640f36be6654b37317bdb676c7f..a0e6204b34c60b69068ff816bbb06090caaafed2 100644 (file)
--- a/source/blender/python/generic/bpy_internal_import.c
+++ b/source/blender/python/generic/bpy_internal_import.c
@@ -226,7 +226,7 @@ PyObject *bpy_text_reimport(PyObject *module, int *found)
 {
        Text *text;
        const char *name;
-       char *filepath;
+       const char *filepath;
 //XXX  Main *maggie = bpy_import_main ? bpy_import_main:G.main;
        Main *maggie = bpy_import_main;
        
@@ -265,7 +265,7 @@ PyObject *bpy_text_reimport(PyObject *module, int *found)
 static PyObject *blender_import(PyObject *UNUSED(self), PyObject *args, PyObject *kw)
 {
        PyObject *exception, *err, *tb;
-       char *name;
+       const char *name;
        int found = 0;
        PyObject *globals = NULL, *locals = NULL, *fromlist = NULL;
        int level = 0; /* relative imports */

diff --git a/source/blender/python/generic/idprop_py_api.c b/source/blender/python/generic/idprop_py_api.c
index 5821d48b99417e2f27045e4360e0ab88a0c2904c..54fed9e11efcdf10f6de7e54d3a2dd61e6db70c2 100644 (file)
--- a/source/blender/python/generic/idprop_py_api.c
+++ b/source/blender/python/generic/idprop_py_api.c
@@ -599,7 +599,7 @@ static PyObject *BPy_IDGroup_MapDataToPy(IDProperty *prop)
                        switch (prop->subtype) {
                                case IDP_FLOAT:
                                {
-                                       float *array = (float *)IDP_Array(prop);
+                                       const float *array = (float *)IDP_Array(prop);
                                        for (i = 0; i < prop->len; i++) {
                                                PyList_SET_ITEM(seq, i, PyFloat_FromDouble(array[i]));
                                        }
@@ -607,7 +607,7 @@ static PyObject *BPy_IDGroup_MapDataToPy(IDProperty *prop)
                                }
                                case IDP_DOUBLE:
                                {
-                                       double *array = (double *)IDP_Array(prop);
+                                       const double *array = (double *)IDP_Array(prop);
                                        for (i = 0; i < prop->len; i++) {
                                                PyList_SET_ITEM(seq, i, PyFloat_FromDouble(array[i]));
                                        }
@@ -615,7 +615,7 @@ static PyObject *BPy_IDGroup_MapDataToPy(IDProperty *prop)
                                }
                                case IDP_INT:
                                {
-                                       int *array = (int *)IDP_Array(prop);
+                                       const int *array = (int *)IDP_Array(prop);
                                        for (i = 0; i < prop->len; i++) {
                                                PyList_SET_ITEM(seq, i, PyLong_FromLong(array[i]));
                                        }
@@ -881,7 +881,7 @@ static PyObject *BPy_IDGroup_clear(BPy_IDProperty *self)
 static PyObject *BPy_IDGroup_Get(BPy_IDProperty *self, PyObject *args)
 {
        IDProperty *idprop;
-       char *key;
+       const char *key;
        PyObject *def = Py_None;
 
        if (!PyArg_ParseTuple(args, "s|O:get", &key, &def))
@@ -1158,7 +1158,7 @@ static PyObject *BPy_IDArray_slice(BPy_IDArray *self, int begin, int end)
        switch (prop->subtype) {
                case IDP_FLOAT:
                {
-                       float *array = (float *)IDP_Array(prop);
+                       const float *array = (float *)IDP_Array(prop);
                        for (count = begin; count < end; count++) {
                                PyTuple_SET_ITEM(tuple, count - begin, PyFloat_FromDouble(array[count]));
                        }
@@ -1166,7 +1166,7 @@ static PyObject *BPy_IDArray_slice(BPy_IDArray *self, int begin, int end)
                }
                case IDP_DOUBLE:
                {
-                       double *array = (double *)IDP_Array(prop);
+                       const double *array = (double *)IDP_Array(prop);
                        for (count = begin; count < end; count++) {
                                PyTuple_SET_ITEM(tuple, count - begin, PyFloat_FromDouble(array[count]));
                        }
@@ -1174,7 +1174,7 @@ static PyObject *BPy_IDArray_slice(BPy_IDArray *self, int begin, int end)
                }
                case IDP_INT:
                {
-                       int *array = (int *)IDP_Array(prop);
+                       const int *array = (int *)IDP_Array(prop);
                        for (count = begin; count < end; count++) {
                                PyTuple_SET_ITEM(tuple, count - begin, PyLong_FromLong(array[count]));
                        }

diff --git a/source/blender/python/generic/py_capi_utils.c b/source/blender/python/generic/py_capi_utils.c
index a37933d027ee8b18cd8f53e0d1de4c87d17fc040..6f72a291a362131b25c338fc870c635140a651ea 100644 (file)
--- a/source/blender/python/generic/py_capi_utils.c
+++ b/source/blender/python/generic/py_capi_utils.c
@@ -284,7 +284,7 @@ PyObject *PyC_Object_GetAttrStringArgs(PyObject *o, Py_ssize_t n, ...)
 {
        Py_ssize_t i;
        PyObject *item = o;
-       char *attr;
+       const char *attr;
        
        va_list vargs;
 
@@ -647,7 +647,7 @@ void PyC_RunQuicky(const char *filepath, int n, ...)
 
                va_start(vargs, n);
                for (i = 0; i * 2 < n; i++) {
-                       char *format = va_arg(vargs, char *);
+                       const char *format = va_arg(vargs, char *);
                        void *ptr = va_arg(vargs, void *);
 
                        ret = PyObject_CallFunction(calcsize, (char *)"s", format);
@@ -704,7 +704,7 @@ void PyC_RunQuicky(const char *filepath, int n, ...)
                                /* now get the values back */
                                va_start(vargs, n);
                                for (i = 0; i * 2 < n; i++) {
-                                       char *format = va_arg(vargs, char *);
+                                       const char *format = va_arg(vargs, char *);
                                        void *ptr = va_arg(vargs, void *);
                                        
                                        PyObject *item;

diff --git a/source/blender/python/intern/bpy.c b/source/blender/python/intern/bpy.c
index ae57588985d9bf982dca25ac4d58fe6b0d05f009..89104821cb0f01088fd4a654f1b3dab6e3f468c2 100644 (file)
--- a/source/blender/python/intern/bpy.c
+++ b/source/blender/python/intern/bpy.c
@@ -144,8 +144,8 @@ static PyObject *bpy_blend_paths(PyObject *UNUSED(self), PyObject *args, PyObjec
 // PyDoc_STRVAR(bpy_user_resource_doc[] = // now in bpy/utils.py
 static PyObject *bpy_user_resource(PyObject *UNUSED(self), PyObject *args, PyObject *kw)
 {
-       char *type;
-       char *subdir = NULL;
+       const char *type;
+       const char *subdir = NULL;
        int folder_id;
        static const char *kwlist[] = {"type", "subdir", NULL};
 
@@ -189,7 +189,7 @@ PyDoc_STRVAR(bpy_resource_path_doc,
 );
 static PyObject *bpy_resource_path(PyObject *UNUSED(self), PyObject *args, PyObject *kw)
 {
-       char *type;
+       const char *type;
        int major = BLENDER_VERSION / 100, minor = BLENDER_VERSION % 100;
        static const char *kwlist[] = {"type", "major", "minor", NULL};
        int folder_id;

diff --git a/source/blender/python/intern/bpy_operator.c b/source/blender/python/intern/bpy_operator.c
index e867aa0095865968d05404c431ce07f6a55a28f7..2c526601fcd2097ce65c460093a4b87626d7490d 100644 (file)
--- a/source/blender/python/intern/bpy_operator.c
+++ b/source/blender/python/intern/bpy_operator.c
@@ -65,10 +65,10 @@
 static PyObject *pyop_poll(PyObject *UNUSED(self), PyObject *args)
 {
        wmOperatorType *ot;
-       char     *opname;
+       const char *opname;
        PyObject *context_dict = NULL; /* optional args */
        PyObject *context_dict_back;
-       char     *context_str = NULL;
+       const char *context_str = NULL;
        PyObject *ret;
 
        int context = WM_OP_EXEC_DEFAULT;
@@ -139,8 +139,8 @@ static PyObject *pyop_call(PyObject *UNUSED(self), PyObject *args)
        PointerRNA ptr;
        int operator_ret = OPERATOR_CANCELLED;
 
-       char     *opname;
-       char     *context_str = NULL;
+       const char *opname;
+       const char *context_str = NULL;
        PyObject *kw = NULL; /* optional args */
        PyObject *context_dict = NULL; /* optional args */
        PyObject *context_dict_back;
@@ -310,7 +310,7 @@ static PyObject *pyop_as_string(PyObject *UNUSED(self), PyObject *args)
        wmOperatorType *ot;
        PointerRNA ptr;
 
-       char     *opname;
+       const char *opname;
        PyObject *kw = NULL; /* optional args */
        int all_args = 1;
        int macro_args = 1;

diff --git a/source/blender/python/intern/bpy_operator_wrap.c b/source/blender/python/intern/bpy_operator_wrap.c
index efee9b13d58acd42b8996a0761f429e0d368983a..11e27ca3e3c3868d30198299d1f9fcef63300a3e 100644 (file)
--- a/source/blender/python/intern/bpy_operator_wrap.c
+++ b/source/blender/python/intern/bpy_operator_wrap.c
@@ -163,7 +163,7 @@ PyObject *PYOP_wrap_macro_define(PyObject *UNUSED(self), PyObject *args)
        PointerRNA ptr_otmacro;
        StructRNA *srna;
 
-       char *opname;
+       const char *opname;
        const char *macroname;
 
        if (!PyArg_ParseTuple(args, "Os:_bpy.ops.macro_define", &macro, &opname))

diff --git a/source/blender/python/intern/bpy_props.c b/source/blender/python/intern/bpy_props.c
index b2f9547b81addb1ccabde734a5b752cfc31b1499..6e36680ec4a80c60b091a96035e68658f194eac3 100644 (file)
--- a/source/blender/python/intern/bpy_props.c
+++ b/source/blender/python/intern/bpy_props.c
@@ -1881,7 +1881,7 @@ static PyObject *BPy_BoolProperty(PyObject *self, PyObject *args, PyObject *kw)
                PropertyRNA *prop;
                PyObject *pyopts = NULL;
                int opts = 0;
-               char *pysubtype = NULL;
+               const char *pysubtype = NULL;
                int subtype = PROP_NONE;
                PyObject *update_cb = NULL;
                PyObject *get_cb = NULL;
@@ -1967,7 +1967,7 @@ static PyObject *BPy_BoolVectorProperty(PyObject *self, PyObject *args, PyObject
                PyObject *pydef = NULL;
                PyObject *pyopts = NULL;
                int opts = 0;
-               char *pysubtype = NULL;
+               const char *pysubtype = NULL;
                int subtype = PROP_NONE;
                PyObject *update_cb = NULL;
                PyObject *get_cb = NULL;
@@ -2062,7 +2062,7 @@ static PyObject *BPy_IntProperty(PyObject *self, PyObject *args, PyObject *kw)
                PropertyRNA *prop;
                PyObject *pyopts = NULL;
                int opts = 0;
-               char *pysubtype = NULL;
+               const char *pysubtype = NULL;
                int subtype = PROP_NONE;
                PyObject *update_cb = NULL;
                PyObject *get_cb = NULL;
@@ -2154,7 +2154,7 @@ static PyObject *BPy_IntVectorProperty(PyObject *self, PyObject *args, PyObject
                PyObject *pydef = NULL;
                PyObject *pyopts = NULL;
                int opts = 0;
-               char *pysubtype = NULL;
+               const char *pysubtype = NULL;
                int subtype = PROP_NONE;
                PyObject *update_cb = NULL;
                PyObject *get_cb = NULL;
@@ -2259,9 +2259,9 @@ static PyObject *BPy_FloatProperty(PyObject *self, PyObject *args, PyObject *kw)
                PropertyRNA *prop;
                PyObject *pyopts = NULL;
                int opts = 0;
-               char *pysubtype = NULL;
+               const char *pysubtype = NULL;
                int subtype = PROP_NONE;
-               char *pyunit = NULL;
+               const char *pyunit = NULL;
                int unit = PROP_UNIT_NONE;
                PyObject *update_cb = NULL;
                PyObject *get_cb = NULL;
@@ -2365,9 +2365,9 @@ static PyObject *BPy_FloatVectorProperty(PyObject *self, PyObject *args, PyObjec
                PyObject *pydef = NULL;
                PyObject *pyopts = NULL;
                int opts = 0;
-               char *pysubtype = NULL;
+               const char *pysubtype = NULL;
                int subtype = PROP_NONE;
-               char *pyunit = NULL;
+               const char *pyunit = NULL;
                int unit = PROP_UNIT_NONE;
                PyObject *update_cb = NULL;
                PyObject *get_cb = NULL;
@@ -2468,7 +2468,7 @@ static PyObject *BPy_StringProperty(PyObject *self, PyObject *args, PyObject *kw
                PropertyRNA *prop;
                PyObject *pyopts = NULL;
                int opts = 0;
-               char *pysubtype = NULL;
+               const char *pysubtype = NULL;
                int subtype = PROP_NONE;
                PyObject *update_cb = NULL;
                PyObject *get_cb = NULL;
@@ -2847,7 +2847,7 @@ static PyObject *BPy_RemoveProperty(PyObject *self, PyObject *args, PyObject *kw
        else {
                static const char *kwlist[] = {"attr", NULL};
                
-               char *id = NULL;
+               const char *id = NULL;
 
                if (!PyArg_ParseTupleAndKeywords(args, kw,
                                                 "s:RemoveProperty",

diff --git a/source/blender/python/intern/bpy_rna.c b/source/blender/python/intern/bpy_rna.c
index cf13efae60e0f148d56a4f0b2ff639347299ebcc..e0c8df71e4fc260ab22c178bf1d5a3cfbf2bea05 100644 (file)
--- a/source/blender/python/intern/bpy_rna.c
+++ b/source/blender/python/intern/bpy_rna.c
@@ -2171,7 +2171,7 @@ static int pyrna_prop_collection_subscript_str_lib_pair_ptr(BPy_PropertyRNA *sel
                                                             PointerRNA *r_ptr
                                                             )
 {
-       char *keyname;
+       const char *keyname;
 
        /* first validate the args, all we know is that they are a tuple */
        if (PyTuple_GET_SIZE(key) != 2) {
@@ -4883,7 +4883,7 @@ static PyObject *pyrna_param_to_py(PointerRNA *ptr, PropertyRNA *prop, void *dat
                                break;
                        case PROP_STRING:
                        {
-                               char *data_ch;
+                               const char *data_ch;
                                PyObject *value_coerce = NULL;
                                const int subtype = RNA_property_subtype(prop);
 

diff --git a/source/blender/python/intern/bpy_rna_callback.c b/source/blender/python/intern/bpy_rna_callback.c
index 9aefe5a733f18fd887a5f64ca2231208d31d1b8e..87c3a6eb4ef06d0b9ee7f7c4ff1af7479c1c6794 100644 (file)
--- a/source/blender/python/intern/bpy_rna_callback.c
+++ b/source/blender/python/intern/bpy_rna_callback.c
@@ -182,8 +182,8 @@ PyObject *pyrna_callback_classmethod_add(PyObject *UNUSED(self), PyObject *args)
        void *handle;
        PyObject *cls;
        PyObject *cb_func, *cb_args;
-       char *cb_regiontype_str;
-       char *cb_event_str;
+       const char *cb_regiontype_str;
+       const char *cb_event_str;
        int cb_event;
        int cb_regiontype;
        StructRNA *srna;
@@ -248,7 +248,7 @@ PyObject *pyrna_callback_classmethod_remove(PyObject *UNUSED(self), PyObject *ar
        void *handle;
        void *customdata;
        StructRNA *srna;
-       char *cb_regiontype_str;
+       const char *cb_regiontype_str;
        int cb_regiontype;
 
        if (PyTuple_GET_SIZE(args) < 2) {

diff --git a/source/blender/python/intern/bpy_util.c b/source/blender/python/intern/bpy_util.c
index b0b0f3469447ac22d6b73a8293880330139bc45f..93183a4f3206ce930961466169cfa6821a88bf03 100644 (file)
--- a/source/blender/python/intern/bpy_util.c
+++ b/source/blender/python/intern/bpy_util.c
@@ -86,7 +86,7 @@ short BPy_errors_to_report(ReportList *reports)
 {
        PyObject *pystring;
        PyObject *pystring_format = NULL;  /* workaround, see below */
-       char *cstring;
+       const char *cstring;
 
        const char *filename;
        int lineno;

diff --git a/source/blender/python/mathutils/mathutils_geometry.c b/source/blender/python/mathutils/mathutils_geometry.c
index 142e5cadfd1a1a7f0a9a87e8ea31247aefd2b4e8..139764abb00a6dd50627f4f9d9a84930738c473e 100644 (file)
--- a/source/blender/python/mathutils/mathutils_geometry.c
+++ b/source/blender/python/mathutils/mathutils_geometry.c
@@ -269,7 +269,7 @@ static PyObject *M_Geometry_intersect_sphere_sphere_2d(PyObject *UNUSED(self), P
 {
        PyObject *ret;
        VectorObject *vec_a, *vec_b;
-       float *v_a, *v_b;
+       const float *v_a, *v_b;
        float rad_a, rad_b;
        float v_ab[2];
        float dist;