code cleanup: use 'const float *' when getting the 3d cursor and not editing it.
authorCampbell Barton <ideasman42@gmail.com>
Mon, 17 Dec 2012 05:38:50 +0000 (05:38 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Mon, 17 Dec 2012 05:38:50 +0000 (05:38 +0000)
source/blender/blenkernel/BKE_main.h
source/blender/editors/armature/editarmature.c
source/blender/editors/gpencil/gpencil_edit.c
source/blender/editors/gpencil/gpencil_paint.c
source/blender/editors/mesh/editmesh_tools.c
source/blender/editors/object/object_add.c
source/blender/editors/space_view3d/view3d_edit.c
source/blender/editors/transform/transform.c
source/blender/editors/transform/transform_generics.c
source/blender/editors/uvedit/uvedit_unwrap_ops.c
source/blender/makesrna/intern/rna_space.c

index b49c5fda475358d2be5420df5bc9b8ecf4cf9e98..da0cba422c3e3ff7491bdba9a320b5d1b5f079d4 100644 (file)
@@ -51,7 +51,7 @@ struct Library;
 typedef struct Main {
        struct Main *next, *prev;
        char name[1024]; /* 1024 = FILE_MAX */
-       short versionfile, subversionfile;
+       short versionfile, subversionfile;  /* see BLENDER_VERSION, BLENDER_SUBVERSION */
        short minversionfile, minsubversionfile;
        int revision;           /* svn revision of binary that saved file */
        short recovered;        /* indicate the main->name (file) is the recovered one */
@@ -92,6 +92,8 @@ typedef struct Main {
        char id_tag_update[256];
 } Main;
 
+#define MAIN_VERSION_ATLEAST(main, ver, subver) \
+       ((main)->versionfile >= (ver) || (main->versionfile == (ver) && (main)->subversionfile >= (subver)))
 
 #ifdef __cplusplus
 }
index e801d3689e53f3868e75894b717adc551c2ce337..ffe58be013955273d5f41d6641654b9eb9e59e5f 100644 (file)
@@ -2062,7 +2062,7 @@ static int armature_calc_roll_exec(bContext *C, wmOperator *op)
                Scene *scene = CTX_data_scene(C);
                View3D *v3d = CTX_wm_view3d(C); /* can be NULL */
                float cursor_local[3];
-               float   *cursor = give_cursor(scene, v3d);
+               const float   *cursor = give_cursor(scene, v3d);
        
 
                copy_v3_v3(cursor_local, cursor);
@@ -2323,7 +2323,8 @@ static int armature_click_extrude_exec(bContext *C, wmOperator *UNUSED(op))
        View3D *v3d;
        bArmature *arm;
        EditBone *ebone, *newbone, *flipbone;
-       float *curs, mat[3][3], imat[3][3];
+       float mat[3][3], imat[3][3];
+       const float *curs;
        int a, to_root = 0;
        Object *obedit;
        Scene *scene;
@@ -2418,7 +2419,7 @@ static int armature_click_extrude_invoke(bContext *C, wmOperator *op, wmEvent *e
        Scene *scene;
        ARegion *ar;
        View3D *v3d;
-       float *fp = NULL, tvec[3], oldcurs[3], mval_f[2];
+       float *fp, tvec[3], oldcurs[3], mval_f[2];
        int retv;
 
        scene = CTX_data_scene(C);
index e9ca7392752b69d4cc3993a1f695f20ad5d40079..e4e640eeefc125497a3471e4a37a07d81bb46600 100644 (file)
@@ -451,7 +451,7 @@ static void gp_strokepoint_convertcoords(bContext *C, bGPDstroke *gps, bGPDspoin
                copy_v3_v3(p3d, &pt->x);
        }
        else {
-               float *fp = give_cursor(scene, v3d);
+               const float *fp = give_cursor(scene, v3d);
                float mvalf[2];
                
                /* get screen coordinate */
index 8fdca730674019ac533ef52fd8f7c223fe27ef03..c40312758fc1d569eb4a43ef6c99be4a1e83bf3b 100644 (file)
@@ -208,7 +208,7 @@ static int gpencil_project_check(tGPsdata *p)
 static void gp_get_3d_reference(tGPsdata *p, float vec[3])
 {
        View3D *v3d = p->sa->spacedata.first;
-       float *fp = give_cursor(p->scene, v3d);
+       const float *fp = give_cursor(p->scene, v3d);
        
        /* the reference point used depends on the owner... */
 #if 0 /* XXX: disabled for now, since we can't draw relative to the owner yet */
index ec9da47c9ac095d914f75e1b5b3886e26395e8e4..be51928fef2f88a2100f528afcd9340b173fcaf7 100644 (file)
@@ -906,7 +906,7 @@ static int edbm_dupli_extrude_cursor_invoke(bContext *C, wmOperator *op, wmEvent
                              BM_ELEM_SELECT, min);
        }
        else {
-               float *curs = give_cursor(vc.scene, vc.v3d);
+               const float *curs = give_cursor(vc.scene, vc.v3d);
                BMOperator bmop;
                BMOIter oiter;
                
@@ -2033,7 +2033,8 @@ static int merge_target(BMEditMesh *em, Scene *scene, View3D *v3d, Object *ob,
 {
        BMIter iter;
        BMVert *v;
-       float *vco = NULL, co[3], cent[3] = {0.0f, 0.0f, 0.0f};
+       float co[3], cent[3] = {0.0f, 0.0f, 0.0f};
+       const float *vco = NULL;
 
        if (target) {
                vco = give_cursor(scene, v3d);
index b3c0368adfe928d70dc2d934aa52d37d597ea2cd..ac1dd2464f8ca3e8aa05f89336b0687331b8f14b 100644 (file)
@@ -146,7 +146,7 @@ void ED_object_location_from_view(bContext *C, float loc[3])
 {
        View3D *v3d = CTX_wm_view3d(C);
        Scene *scene = CTX_data_scene(C);
-       float *cursor;
+       const float *cursor;
 
        cursor = give_cursor(scene, v3d);
 
index cd5770ba940b528a192bc8aa5dd71012c6fce86d..a1f0bf6949743b1d0d8e073e0d1f8475aca33ec2 100644 (file)
@@ -3683,10 +3683,9 @@ static int view3d_cursor3d_invoke(bContext *C, wmOperator *UNUSED(op), wmEvent *
        ARegion *ar = CTX_wm_region(C);
        View3D *v3d = CTX_wm_view3d(C);
        RegionView3D *rv3d = CTX_wm_region_view3d(C);
-       float *fp = NULL;
+       float *fp = give_cursor(scene, v3d);
        float mval_fl[2];
        int flip;
-       fp = give_cursor(scene, v3d);
 
        flip = initgrabz(rv3d, fp[0], fp[1], fp[2]);
        
index 6a5904767b737f5e6e99706b154c750c4b6d72cd..e85fe005670861cf84478d4b6aeef9b299d8e19c 100644 (file)
@@ -2579,7 +2579,8 @@ int handleEventWarp(TransInfo *t, wmEvent *event)
 int Warp(TransInfo *t, const int UNUSED(mval[2]))
 {
        TransData *td = t->data;
-       float vec[3], circumfac, dist, phi0, co, si, *curs, cursor[3], gcursor[3];
+       float vec[3], circumfac, dist, phi0, co, si, cursor[3], gcursor[3];
+       const float *curs;
        int i;
        char str[50];
        
index 615bb786071bb8233af9a756bbf94ed292dcd12a..6d7ddd4e0edeff263d9f829bfec7a866982fdd7b 100644 (file)
@@ -1445,7 +1445,7 @@ void calculateCenter2D(TransInfo *t)
 
 void calculateCenterCursor(TransInfo *t)
 {
-       float *cursor;
+       const float *cursor;
        
        cursor = give_cursor(t->scene, t->view);
        copy_v3_v3(t->center, cursor);
index cb54689a3c0314506b96312e0030cf189ff10a02..2ca711a4a6a5f89850e6278cd30560fc60dfec34 100644 (file)
@@ -895,16 +895,18 @@ void ED_uvedit_live_unwrap(Scene *scene, Object *obedit)
 static void uv_map_transform_center(Scene *scene, View3D *v3d, float *result, 
                                     Object *ob, BMEditMesh *em)
 {
-       BMFace *efa;
-       BMLoop *l;
-       BMIter iter, liter;
-       float min[3], max[3], *cursx;
        int around = (v3d) ? v3d->around : V3D_CENTER;
 
        /* only operates on the edit object - this is all that's needed now */
 
        switch (around) {
                case V3D_CENTER: /* bounding box center */
+               {
+                       BMFace *efa;
+                       BMLoop *l;
+                       BMIter iter, liter;
+                       float min[3], max[3];
+
                        INIT_MINMAX(min, max);
                        
                        BM_ITER_MESH (efa, &iter, em->bm, BM_FACES_OF_MESH) {
@@ -916,17 +918,18 @@ static void uv_map_transform_center(Scene *scene, View3D *v3d, float *result,
                        }
                        mid_v3_v3v3(result, min, max);
                        break;
-
+               }
                case V3D_CURSOR:  /* cursor center */
-                       cursx = give_cursor(scene, v3d);
+               {
+                       const float *curs = give_cursor(scene, v3d);
                        /* shift to objects world */
-                       sub_v3_v3v3(result, cursx, ob->obmat[3]);
+                       sub_v3_v3v3(result, curs, ob->obmat[3]);
                        break;
-
+               }
                case V3D_LOCAL:     /* object center */
                case V3D_CENTROID:  /* multiple objects centers, only one object here*/
                default:
-                       result[0] = result[1] = result[2] = 0.0;
+                       zero_v3(result);
                        break;
        }
 }
index a59cfc46c3734360287f4987aef8e1c35ef78c9d..f7475151e9de4cd3a99666d03bd32436d1203c57 100644 (file)
@@ -321,7 +321,7 @@ static void rna_View3D_CursorLocation_get(PointerRNA *ptr, float *values)
        View3D *v3d = (View3D *)(ptr->data);
        bScreen *sc = (bScreen *)ptr->id.data;
        Scene *scene = (Scene *)sc->scene;
-       float *loc = give_cursor(scene, v3d);
+       const float *loc = give_cursor(scene, v3d);
        
        copy_v3_v3(values, loc);
 }