Code Cleanup:
authorCampbell Barton <ideasman42@gmail.com>
Tue, 7 Feb 2012 14:01:34 +0000 (14:01 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Tue, 7 Feb 2012 14:01:34 +0000 (14:01 +0000)
BMesh  operator flag was being passed around as in int, but the value is infact a short. use short for all functions now.

source/blender/bmesh/bmesh_operator_api.h
source/blender/bmesh/intern/bmesh_construct.c
source/blender/bmesh/intern/bmesh_operators.c
source/blender/bmesh/intern/bmesh_polygon.c
source/blender/bmesh/intern/bmesh_private.h

index 9bd73873606f48c14fe966fc481af9dd8d71eefd..2def4fd928458448d9b26b05e2f0c8814a3fb441 100644 (file)
@@ -181,13 +181,13 @@ void BMO_Finish_Op(struct BMesh *bm, struct BMOperator *op);
 #define BMO_ToggleFlag(bm, element, oflag) (((BMHeader*)(element))->flags[bm->stackdepth-1].f ^= (oflag))
 
 /*profiling showed a significant amount of time spent in BMO_TestFlag
-void BMO_SetFlag(struct BMesh *bm, void *element, int oflag);
-void BMO_ClearFlag(struct BMesh *bm, void *element, int oflag);
-int BMO_TestFlag(struct BMesh *bm, void *element, int oflag);*/
+void BMO_SetFlag(struct BMesh *bm, void *element, const short oflag);
+void BMO_ClearFlag(struct BMesh *bm, void *element, const short oflag);
+int BMO_TestFlag(struct BMesh *bm, void *element, const short oflag);*/
 
 /* count the number of elements with a specific flag.
  * type can be a bitmask of BM_FACE, BM_EDGE, or BM_FACE. */
-int BMO_CountFlag(struct BMesh *bm, const int oflag, const char htype);
+int BMO_CountFlag(struct BMesh *bm, const short oflag, const char htype);
 
 /*---------formatted operator initialization/execution-----------*/
 /*
@@ -251,13 +251,13 @@ BMOpSlot *BMO_GetSlot(struct BMOperator *op, const char *slotname);
 void BMO_CopySlot(struct BMOperator *source_op, struct BMOperator *dest_op,
                   const char *src, const char *dst);
 
-/*remove tool flagged elements*/
-void BM_remove_tagged_faces(struct BMesh *bm, const int oflag);
-void BM_remove_tagged_edges(struct BMesh *bm, const int oflag);
-void BM_remove_tagged_verts(struct BMesh *bm, const int oflag);
+/* remove tool flagged elements */
+void BM_remove_tagged_faces(struct BMesh *bm, const short oflag);
+void BM_remove_tagged_edges(struct BMesh *bm, const short oflag);
+void BM_remove_tagged_verts(struct BMesh *bm, const short oflag);
 
-void BMO_Set_OpFlag(struct BMesh *bm, struct BMOperator *op, const int oflag);
-void BMO_Clear_OpFlag(struct BMesh *bm, struct BMOperator *op, const int oflag);
+void BMO_Set_OpFlag(struct BMesh *bm, struct BMOperator *op, const int op_flag);
+void BMO_Clear_OpFlag(struct BMesh *bm, struct BMOperator *op, const int op_flag);
 
 void BMO_Set_Float(struct BMOperator *op, const char *slotname, float f);
 float BMO_Get_Float(BMOperator *op, const char *slotname);
@@ -282,19 +282,19 @@ void BMO_Set_Mat(struct BMOperator *op, const char *slotname, float *mat, int si
 void BMO_Get_Mat4(struct BMOperator *op, const char *slotname, float mat[4][4]);
 void BMO_Get_Mat3(struct BMOperator *op, const char *slotname, float mat[3][3]);
 
-void BMO_Clear_Flag_All(BMesh *bm, BMOperator *op, const char htype, const int oflag);
+void BMO_Clear_Flag_All(BMesh *bm, BMOperator *op, const char htype, const short oflag);
 
 /* puts every element of type type (which is a bitmask) with tool flag flag,
  * into a slot. */
 void BMO_Flag_To_Slot(struct BMesh *bm, struct BMOperator *op, const char *slotname,
-                      const int oflag, const char htype);
+                      const short oflag, const char htype);
 
 /* tool-flags all elements inside an element slot array with flag flag. */
 void BMO_Flag_Buffer(struct BMesh *bm, struct BMOperator *op, const char *slotname,
-                     const int oflag, const char htype);
+                     const short oflag, const char htype);
 /* clears tool-flag flag from all elements inside a slot array. */
 void BMO_Unflag_Buffer(struct BMesh *bm, struct BMOperator *op, const char *slotname,
-                       const int oflag, const char htype);
+                       const short oflag, const char htype);
 
 /* tool-flags all elements inside an element slot array with flag flag. */
 void BMO_HeaderFlag_Buffer(struct BMesh *bm, struct BMOperator *op, const char *slotname,
@@ -315,7 +315,7 @@ int BMO_CountSlotMap(struct BMesh *bm, struct BMOperator *op, const char *slotna
 
 /* Counts the number of edges with tool flag toolflag around
  */
-int BMO_Vert_CountEdgeFlags(BMesh *bm, BMVert *v, const int oflag);
+int BMO_Vert_CountEdgeFlags(BMesh *bm, BMVert *v, const short oflag);
 
 /* inserts a key/value mapping into a mapping slot.  note that it copies the
  * value, it doesn't store a reference to it. */
@@ -339,7 +339,7 @@ int BMO_Vert_CountEdgeFlags(BMesh *bm, BMVert *v, const int oflag);
 /* flags all elements in a mapping.  note that the mapping must only have
  * bmesh elements in it.*/
 void BMO_Mapping_To_Flag(struct BMesh *bm, struct BMOperator *op,
-                         const char *slotname, const int oflag);
+                         const char *slotname, const short oflag);
 
 /* pointer versoins of BMO_Get_MapFloat and BMO_Insert_MapFloat.
  *
index 910be3f92aad1eca64a496118ff323cdfd5bc4d9..e2a6aea8498f195e0a6ad11f9c736046ec136ae8 100644 (file)
@@ -368,7 +368,7 @@ err:
  *
  */
 
-void BM_remove_tagged_faces(BMesh *bm, const int oflag)
+void BM_remove_tagged_faces(BMesh *bm, const short oflag)
 {
        BMFace *f;
        BMIter iter;
@@ -380,7 +380,7 @@ void BM_remove_tagged_faces(BMesh *bm, const int oflag)
        }
 }
 
-void BM_remove_tagged_edges(BMesh *bm, const int oflag)
+void BM_remove_tagged_edges(BMesh *bm, const short oflag)
 {
        BMEdge *e;
        BMIter iter;
@@ -392,7 +392,7 @@ void BM_remove_tagged_edges(BMesh *bm, const int oflag)
        }
 }
 
-void BM_remove_tagged_verts(BMesh *bm, const int oflag)
+void BM_remove_tagged_verts(BMesh *bm, const short oflag)
 {
        BMVert *v;
        BMIter iter;
index a88ac28e52adf37587284168c3ba17511df194a6..78cce34053d688265a232205022244f56feca4d5 100644 (file)
@@ -81,14 +81,14 @@ const int BMOP_OPSLOT_TYPEINFO[] = {
 /* Dummy slot so there is something to return when slot name lookup fails */
 static BMOpSlot BMOpEmptySlot = {0};
 
-void BMO_Set_OpFlag(BMesh *UNUSED(bm), BMOperator *op, const int flag)
+void BMO_Set_OpFlag(BMesh *UNUSED(bm), BMOperator *op, const int op_flag)
 {
-       op->flag |= flag;
+       op->flag |= op_flag;
 }
 
-void BMO_Clear_OpFlag(BMesh *UNUSED(bm), BMOperator *op, const int flag)
+void BMO_Clear_OpFlag(BMesh *UNUSED(bm), BMOperator *op, const int op_flag)
 {
-       op->flag &= ~flag;
+       op->flag &= ~op_flag;
 }
 
 /*
@@ -433,7 +433,7 @@ void BMO_Get_Vec(BMOperator *op, const char *slotname, float r_vec[3])
  *
  */
 
-int BMO_CountFlag(BMesh *bm, const int oflag, const char htype)
+int BMO_CountFlag(BMesh *bm, const short oflag, const char htype)
 {
        BMIter elements;
        BMHeader *e;
@@ -461,7 +461,7 @@ int BMO_CountFlag(BMesh *bm, const int oflag, const char htype)
        return count;
 }
 
-void BMO_Clear_Flag_All(BMesh *bm, BMOperator *UNUSED(op), const char htype, const int oflag)
+void BMO_Clear_Flag_All(BMesh *bm, BMOperator *UNUSED(op), const char htype, const short oflag)
 {
        const char iter_types[3] = {BM_VERTS_OF_MESH,
                                    BM_EDGES_OF_MESH,
@@ -540,7 +540,7 @@ void *BMO_Grow_Array(BMesh *bm, BMOperator *op, int slotcode, int totadd)
 #endif
 
 void BMO_Mapping_To_Flag(struct BMesh *bm, struct BMOperator *op,
-                         const char *slotname, const int oflag)
+                         const char *slotname, const short oflag)
 {
        GHashIterator it;
        BMOpSlot *slot = BMO_GetSlot(op, slotname);
@@ -673,7 +673,7 @@ void BMO_HeaderFlag_To_Slot(BMesh *bm, BMOperator *op, const char *slotname,
  * into an output slot for an operator.
  */
 void BMO_Flag_To_Slot(BMesh *bm, BMOperator *op, const char *slotname,
-                      const int oflag, const char htype)
+                      const short oflag, const char htype)
 {
        BMIter elements;
        BMHeader *e;
@@ -765,7 +765,7 @@ void BMO_UnHeaderFlag_Buffer(BMesh *bm, BMOperator *op, const char *slotname,
                BM_ClearHFlag(data[i], hflag);
        }
 }
-int BMO_Vert_CountEdgeFlags(BMesh *bm, BMVert *v, const int oflag)
+int BMO_Vert_CountEdgeFlags(BMesh *bm, BMVert *v, const short oflag)
 {
        int count = 0;
 
@@ -791,7 +791,7 @@ int BMO_Vert_CountEdgeFlags(BMesh *bm, BMVert *v, const int oflag)
  * Flags elements in a slots buffer
  */
 void BMO_Flag_Buffer(BMesh *bm, BMOperator *op, const char *slotname,
-                     const int oflag, const char htype)
+                     const short oflag, const char htype)
 {
        BMOpSlot *slot = BMO_GetSlot(op, slotname);
        BMHeader **data =  slot->data.p;
@@ -812,7 +812,7 @@ void BMO_Flag_Buffer(BMesh *bm, BMOperator *op, const char *slotname,
  * Removes flags from elements in a slots buffer
  */
 void BMO_Unflag_Buffer(BMesh *bm, BMOperator *op, const char *slotname,
-                       const int oflag, const char htype)
+                       const short oflag, const char htype)
 {
        BMOpSlot *slot = BMO_GetSlot(op, slotname);
        BMHeader **data =  slot->data.p;
@@ -1407,7 +1407,7 @@ int BMO_CallOpf(BMesh *bm, const char *fmt, ...)
 #ifdef BMO_ToggleFlag
 #undef BMO_ToggleFlag
 #endif
-static void BMO_ToggleFlag(BMesh *bm, void *element, int oflag)
+static void BMO_ToggleFlag(BMesh *bm, void *element, const short oflag)
 {
        BMHeader *head = element;
        head->flags[bm->stackdepth - 1].f ^= oflag;
@@ -1421,7 +1421,7 @@ static void BMO_ToggleFlag(BMesh *bm, void *element, int oflag)
 #ifdef BMO_SetFlag
 #undef BMO_SetFlag
 #endif
-static void BMO_SetFlag(BMesh *bm, void *element, const int oflag)
+static void BMO_SetFlag(BMesh *bm, void *element, const short oflag)
 {
        BMHeader *head = element;
        head->flags[bm->stackdepth - 1].f |= oflag;
@@ -1435,7 +1435,7 @@ static void BMO_SetFlag(BMesh *bm, void *element, const int oflag)
 #ifdef BMO_ClearFlag
 #undef BMO_ClearFlag
 #endif
-static void BMO_ClearFlag(BMesh *bm, void *element, const int oflag)
+static void BMO_ClearFlag(BMesh *bm, void *element, const short oflag)
 {
        BMHeader *head = element;
        head->flags[bm->stackdepth - 1].f &= ~oflag;
@@ -1450,7 +1450,7 @@ static void BMO_ClearFlag(BMesh *bm, void *element, const int oflag)
 #ifdef BMO_TestFlag
 #undef BMO_TestFlag
 #endif
-static int BMO_TestFlag(BMesh *bm, void *element, const int oflag)
+static int BMO_TestFlag(BMesh *bm, void *element, const short oflag)
 {
        BMHeader *head = element;
        if (head->flags[bm->stackdepth - 1].f & oflag)
index 8af4fe2fba1b456986c044bc12e0440d91ad62a5..6ec4faae3ea9e1f199f095a114d3797fa431bc2b 100644 (file)
@@ -851,7 +851,7 @@ static BMLoop *find_ear(BMesh *UNUSED(bm), BMFace *f, float (*verts)[3], const i
  * triangles, and will be NULL-terminated.
  */
 void BM_Triangulate_Face(BMesh *bm, BMFace *f, float (*projectverts)[3],
-                         const int newedge_oflag, const int newface_oflag, BMFace **newfaces)
+                         const short newedge_oflag, const short newface_oflag, BMFace **newfaces)
 {
        int i, done, nvert, nf_i = 0;
        BMLoop *newl, *nextloop;
index 5d45378ea90b5809a11b54aa762c253e8ce06148..381c3ba8d291822c3876cca5333ac0e956ac6bb5 100644 (file)
@@ -82,7 +82,7 @@ int bmesh_get_filter_argtype(int type);
 /* Polygon Utilities ? FIXME... where do these each go? */
 /* newedgeflag sets a flag layer flag, obviously not the header flag. */
 void BM_Triangulate_Face(BMesh *bm, BMFace *f, float (*projectverts)[3],
-                         int newedgeflag, int newfaceflag, BMFace **newfaces);
+                         const short newedge_oflag, const short newface_oflag, BMFace **newfaces);
 void bmesh_update_face_normal(struct BMesh *bm, struct BMFace *f, float no[3],
                               float (*projectverts)[3]);
 void bmesh_update_face_normal_vertex_cos(struct BMesh *bm, struct BMFace *f, float no[3],