Mark context as const when it's possible and avoid typecasts from const bContext...
authorSergey Sharybin <sergey.vfx@gmail.com>
Tue, 19 Jun 2012 14:09:44 +0000 (14:09 +0000)
committerSergey Sharybin <sergey.vfx@gmail.com>
Tue, 19 Jun 2012 14:09:44 +0000 (14:09 +0000)
source/blender/editors/include/ED_clip.h
source/blender/editors/mask/mask_add.c
source/blender/editors/mask/mask_draw.c
source/blender/editors/mask/mask_edit.c
source/blender/editors/mask/mask_intern.h
source/blender/editors/mask/mask_ops.c
source/blender/editors/space_clip/clip_editor.c
source/blender/editors/space_clip/space_clip.c

index 649266beec7bdda6c5745f5c05f09dce89387e0c..9284f547047ed1e7e033267b859aaa22974a9911 100644 (file)
@@ -70,9 +70,9 @@ void ED_clip_update_frame(const struct Main *mainp, int cfra);
 int ED_clip_view_selection(struct SpaceClip *sc, struct ARegion *ar, int fit);
 
 void ED_clip_point_undistorted_pos(SpaceClip * sc, const float co[2], float r_co[2]);
-void ED_clip_point_stable_pos(struct bContext *C, float x, float y, float *xr, float *yr);
-void ED_clip_point_stable_pos__reverse(SpaceClip * sc, ARegion *ar, const float co[2], float r_co[2]);
-void ED_clip_mouse_pos(struct bContext *C, struct wmEvent *event, float co[2]);
+void ED_clip_point_stable_pos(const struct bContext *C, float x, float y, float *xr, float *yr);
+void ED_clip_point_stable_pos__reverse(SpaceClip *sc, ARegion *ar, const float co[2], float r_co[2]);
+void ED_clip_mouse_pos(const struct bContext *C, struct wmEvent *event, float co[2]);
 
 int ED_space_clip_texture_buffer_supported(struct SpaceClip *sc);
 int ED_space_clip_load_movieclip_buffer(struct SpaceClip *sc, struct ImBuf *ibuf);
index 0bc9adb65778c9864a4aa7856aca14a9772bfee9..c9f6dc0c5fb30c2a9fe37e2ff8933ace6046c8a5 100644 (file)
@@ -52,7 +52,7 @@
 #include "mask_intern.h"  /* own include */
 
 
-static int find_nearest_diff_point(bContext *C, Mask *mask, const float normal_co[2], int threshold, int feather,
+static int find_nearest_diff_point(const bContext *C, Mask *mask, const float normal_co[2], int threshold, int feather,
                                    MaskLayer **masklay_r, MaskSpline **spline_r, MaskSplinePoint **point_r,
                                    float *u_r, float tangent[2],
                                    const short use_deform)
@@ -176,7 +176,7 @@ static int find_nearest_diff_point(bContext *C, Mask *mask, const float normal_c
 
 /******************** add vertex *********************/
 
-static void setup_vertex_point(bContext *C, Mask *mask, MaskSpline *spline, MaskSplinePoint *new_point,
+static void setup_vertex_point(const bContext *C, Mask *mask, MaskSpline *spline, MaskSplinePoint *new_point,
                                const float point_co[2], const float tangent[2], const float u,
                                MaskSplinePoint *reference_point, const short reference_adjacent)
 {
@@ -365,7 +365,7 @@ static void mask_spline_add_point_at_index(MaskSpline *spline, int point_index)
        spline->tot_point++;
 }
 
-static int add_vertex_subdivide(bContext *C, Mask *mask, const float co[2])
+static int add_vertex_subdivide(const bContext *C, Mask *mask, const float co[2])
 {
        MaskLayer *masklay;
        MaskSpline *spline;
@@ -399,7 +399,7 @@ static int add_vertex_subdivide(bContext *C, Mask *mask, const float co[2])
        return FALSE;
 }
 
-static int add_vertex_extrude(bContext *C, Mask *mask, MaskLayer *masklay, const float co[2])
+static int add_vertex_extrude(const bContext *C, Mask *mask, MaskLayer *masklay, const float co[2])
 {
        MaskSpline *spline;
        MaskSplinePoint *point;
@@ -500,7 +500,7 @@ static int add_vertex_extrude(bContext *C, Mask *mask, MaskLayer *masklay, const
        return TRUE;
 }
 
-static int add_vertex_new(bContext *C, Mask *mask, MaskLayer *masklay, const float co[2])
+static int add_vertex_new(const bContext *C, Mask *mask, MaskLayer *masklay, const float co[2])
 {
        MaskSpline *spline;
        MaskSplinePoint *point;
index 678186f0e8c3fe803764196b67cc3c0f67d1dfb8..71d3caf74c50b6548ec1c73a19689e89d3b97335 100644 (file)
@@ -441,10 +441,7 @@ void ED_mask_draw(const bContext *C,
        if (!mask)
                return;
 
-       /* TODO: for now, in the future better to make sure all utility functions
-        *       are using const specifier for non-changing pointers
-        */
-       ED_mask_size((bContext *)C, &width, &height);
+       ED_mask_size(C, &width, &height);
 
        draw_masklays(mask, draw_flag, draw_type, width, height);
 }
index 52711c8da550013381762105c8189686ccd923a2..3396f664820d25ca11118a19d132acc6723a0cca 100644 (file)
@@ -76,7 +76,7 @@ int ED_maskedit_mask_poll(bContext *C)
 
 /********************** registration *********************/
 
-void ED_mask_mouse_pos(bContext *C, wmEvent *event, float co[2])
+void ED_mask_mouse_pos(const bContext *C, wmEvent *event, float co[2])
 {
        SpaceClip *sc = CTX_wm_space_clip(C);
 
@@ -92,7 +92,7 @@ void ED_mask_mouse_pos(bContext *C, wmEvent *event, float co[2])
 
 /* input:  x/y   - mval space
  * output: xr/yr - mask point space */
-void ED_mask_point_pos(bContext *C, float x, float y, float *xr, float *yr)
+void ED_mask_point_pos(const bContext *C, float x, float y, float *xr, float *yr)
 {
        SpaceClip *sc = CTX_wm_space_clip(C);
        float co[2];
@@ -110,7 +110,7 @@ void ED_mask_point_pos(bContext *C, float x, float y, float *xr, float *yr)
        *yr = co[1];
 }
 
-void ED_mask_point_pos__reverse(bContext *C, float x, float y, float *xr, float *yr)
+void ED_mask_point_pos__reverse(const bContext *C, float x, float y, float *xr, float *yr)
 {
        SpaceClip *sc = CTX_wm_space_clip(C);
        ARegion *ar = CTX_wm_region(C);
@@ -132,7 +132,7 @@ void ED_mask_point_pos__reverse(bContext *C, float x, float y, float *xr, float
        *yr = co[1];
 }
 
-void ED_mask_size(bContext *C, int *width, int *height)
+void ED_mask_size(const bContext *C, int *width, int *height)
 {
        ScrArea *sa = CTX_wm_area(C);
        if (sa && sa->spacedata.first) {
@@ -154,7 +154,7 @@ void ED_mask_size(bContext *C, int *width, int *height)
        *height = 0;
 }
 
-void ED_mask_aspect(bContext *C, float *aspx, float *aspy)
+void ED_mask_aspect(const bContext *C, float *aspx, float *aspy)
 {
        SpaceClip *sc = CTX_wm_space_clip(C);
 
@@ -168,7 +168,7 @@ void ED_mask_aspect(bContext *C, float *aspx, float *aspy)
        }
 }
 
-void ED_mask_pixelspace_factor(bContext *C, float *scalex, float *scaley)
+void ED_mask_pixelspace_factor(const bContext *C, float *scalex, float *scaley)
 {
        SpaceClip *sc = CTX_wm_space_clip(C);
 
index c34558d2a01e4bdf30d0d2984ebf245f35e22943..408b585bf4c38d000ca4e34bce49d45e27e6690e 100644 (file)
@@ -61,12 +61,12 @@ void MASK_OT_normals_make_consistent(struct wmOperatorType *ot);
 void MASK_OT_handle_type_set(struct wmOperatorType *ot);
 
 int ED_mask_feather_find_nearest(
-        struct bContext *C, struct Mask *mask, float normal_co[2], int threshold,
+        const struct bContext *C, struct Mask *mask, float normal_co[2], int threshold,
         struct MaskLayer **masklay_r, struct MaskSpline **spline_r, struct MaskSplinePoint **point_r,
         struct MaskSplinePointUW **uw_r, float *score);
 
 struct MaskSplinePoint *ED_mask_point_find_nearest(
-        struct bContext *C, struct Mask *mask, float normal_co[2], int threshold,
+        const struct bContext *C, struct Mask *mask, float normal_co[2], int threshold,
         struct MaskLayer **masklay_r, struct MaskSpline **spline_r, int *is_handle_r,
         float *score);
 
@@ -97,14 +97,14 @@ void ED_mask_select_flush_all(struct Mask *mask);
 int ED_maskedit_poll(struct bContext *C);
 int ED_maskedit_mask_poll(struct bContext *C);
 
-void ED_mask_size(struct bContext *C, int *width, int *height);
-void ED_mask_aspect(struct bContext *C, float *aspx, float *aspy);
+void ED_mask_size(const struct bContext *C, int *width, int *height);
+void ED_mask_aspect(const struct bContext *C, float *aspx, float *aspy);
 
-void ED_mask_pixelspace_factor(struct bContext *C, float *scalex, float *scaley);
-void ED_mask_mouse_pos(struct bContext *C, struct wmEvent *event, float co[2]);
+void ED_mask_pixelspace_factor(const struct bContext *C, float *scalex, float *scaley);
+void ED_mask_mouse_pos(const struct bContext *C, struct wmEvent *event, float co[2]);
 
-void ED_mask_point_pos(struct bContext *C, float x, float y, float *xr, float *yr);
-void ED_mask_point_pos__reverse(struct bContext *C, float x, float y, float *xr, float *yr);
+void ED_mask_point_pos(const struct bContext *C, float x, float y, float *xr, float *yr);
+void ED_mask_point_pos__reverse(const struct bContext *C, float x, float y, float *xr, float *yr);
 
 /* mask_shapekey.c */
 void MASK_OT_shape_key_insert(struct wmOperatorType *ot);
index b770e5e9dbac89d8823e2955759c6cae4c64dd05..91ba05c1ab2e02805d316dc9a10d64c20e11956a 100644 (file)
@@ -57,7 +57,7 @@
 
 /******************** utility functions *********************/
 
-MaskSplinePoint *ED_mask_point_find_nearest(bContext *C, Mask *mask, float normal_co[2], int threshold,
+MaskSplinePoint *ED_mask_point_find_nearest(const bContext *C, Mask *mask, float normal_co[2], int threshold,
                                             MaskLayer **masklay_r, MaskSpline **spline_r, int *is_handle_r,
                                             float *score)
 {
@@ -153,7 +153,7 @@ MaskSplinePoint *ED_mask_point_find_nearest(bContext *C, Mask *mask, float norma
        return NULL;
 }
 
-int ED_mask_feather_find_nearest(bContext *C, Mask *mask, float normal_co[2], int threshold,
+int ED_mask_feather_find_nearest(const bContext *C, Mask *mask, float normal_co[2], int threshold,
                                  MaskLayer **masklay_r, MaskSpline **spline_r, MaskSplinePoint **point_r,
                                  MaskSplinePointUW **uw_r, float *score)
 {
index 0bad9f86ea10a54c24827d9a3ee054c7c0d5080d..aeddb24d84d4cea7e01b366b555196acd82f2b3f 100644 (file)
@@ -446,7 +446,7 @@ void ED_clip_point_undistorted_pos(SpaceClip *sc, const float co[2], float r_co[
        }
 }
 
-void ED_clip_point_stable_pos(bContext *C, float x, float y, float *xr, float *yr)
+void ED_clip_point_stable_pos(const bContext *C, float x, float y, float *xr, float *yr)
 {
        ARegion *ar = CTX_wm_region(C);
        SpaceClip *sc = CTX_wm_space_clip(C);
@@ -506,7 +506,7 @@ void ED_clip_point_stable_pos__reverse(SpaceClip *sc, ARegion *ar, const float c
        r_co[1] = (pos[1] * height * zoomy) + (float)sy;
 }
 
-void ED_clip_mouse_pos(bContext *C, wmEvent *event, float co[2])
+void ED_clip_mouse_pos(const bContext *C, wmEvent *event, float co[2])
 {
        ED_clip_point_stable_pos(C, event->mval[0], event->mval[1], &co[0], &co[1]);
 }
index 174b7fcb37390ffe3b95b5b86c1a9edbc28debc7..bca0b2a476d6d8533ff36246e0cb6074ad639495 100644 (file)
@@ -1147,7 +1147,7 @@ static void clip_main_area_draw(const bContext *C, ARegion *ar)
                glScalef(maxdim * zoomx, maxdim * zoomy, 0);
                glMultMatrixf(sc->stabmat);
 
-               ED_mask_draw((bContext *)C, sc->mask_draw_flag, sc->mask_draw_type);
+               ED_mask_draw(C, sc->mask_draw_flag, sc->mask_draw_type);
 
                ED_region_draw_cb_draw(C, ar, REGION_DRAW_POST_VIEW);