diff options
author | Jacques Lucke <jacques@blender.org> | 2020-09-16 13:23:23 +0300 |
---|---|---|
committer | Jacques Lucke <jacques@blender.org> | 2020-09-16 13:23:23 +0300 |
commit | 27a5da4dc3a399e1fe7e88dc8722a891e9cfcf78 (patch) | |
tree | 7ad797f8746a58e527ed301bf7bcab4a389bb687 /source/blender/editors/curve | |
parent | d376aea61840587eddcf75386b804673e5593d60 (diff) |
Cleanup: use uint8_t for various flags in curves
Previously, it was kind of a mess. In different places it was using `char`, `short` and `int`.
The changed properties are flags that are operated upon using bit operations. Therefore, the integer type should be unsigned. Since we only use 2 bits of these flags, `uint8_t` is large enough.
Especially note the change I had to make in `RNA_define.h` to make this work.
Reviewers: campbellbarton
Differential Revision: https://developer.blender.org/D8844
Diffstat (limited to 'source/blender/editors/curve')
-rw-r--r-- | source/blender/editors/curve/curve_intern.h | 8 | ||||
-rw-r--r-- | source/blender/editors/curve/editcurve.c | 10 | ||||
-rw-r--r-- | source/blender/editors/curve/editcurve_query.c | 2 | ||||
-rw-r--r-- | source/blender/editors/curve/editcurve_select.c | 4 |
4 files changed, 12 insertions, 12 deletions
diff --git a/source/blender/editors/curve/curve_intern.h b/source/blender/editors/curve/curve_intern.h index 4426739e421..704e4b740de 100644 --- a/source/blender/editors/curve/curve_intern.h +++ b/source/blender/editors/curve/curve_intern.h @@ -71,8 +71,8 @@ typedef enum eCurveElem_Types { } eCurveElem_Types; /* internal select utils */ -bool select_beztriple(BezTriple *bezt, bool selstatus, short flag, eVisible_Types hidden); -bool select_bpoint(BPoint *bp, bool selstatus, short flag, bool hidden); +bool select_beztriple(BezTriple *bezt, bool selstatus, uint8_t flag, eVisible_Types hidden); +bool select_bpoint(BPoint *bp, bool selstatus, uint8_t flag, bool hidden); void FONT_OT_text_insert(struct wmOperatorType *ot); void FONT_OT_line_break(struct wmOperatorType *ot); @@ -142,8 +142,8 @@ struct GHash *ED_curve_keyindex_hash_duplicate(struct GHash *keyindex); void ED_curve_keyindex_update_nurb(struct EditNurb *editnurb, struct Nurb *nu, struct Nurb *newnu); /* helper functions */ -void ed_editnurb_translate_flag(struct ListBase *editnurb, short flag, const float vec[3]); -bool ed_editnurb_extrude_flag(struct EditNurb *editnurb, const short flag); +void ed_editnurb_translate_flag(struct ListBase *editnurb, uint8_t flag, const float vec[3]); +bool ed_editnurb_extrude_flag(struct EditNurb *editnurb, const uint8_t flag); bool ed_editnurb_spin(float viewmat[4][4], struct View3D *v3d, struct Object *obedit, diff --git a/source/blender/editors/curve/editcurve.c b/source/blender/editors/curve/editcurve.c index e6815582a04..9214e0b283a 100644 --- a/source/blender/editors/curve/editcurve.c +++ b/source/blender/editors/curve/editcurve.c @@ -78,7 +78,7 @@ void selectend_nurb(Object *obedit, enum eEndPoint_Types selfirst, bool doswap, bool selstatus); static void adduplicateflagNurb( - Object *obedit, View3D *v3d, ListBase *newnurb, const short flag, const bool split); + Object *obedit, View3D *v3d, ListBase *newnurb, const uint8_t flag, const bool split); static bool curve_delete_segments(Object *obedit, View3D *v3d, const bool split); static bool curve_delete_vertices(Object *obedit, View3D *v3d); @@ -1600,7 +1600,7 @@ void CURVE_OT_split(wmOperatorType *ot) /** \name Flag Utility Functions * \{ */ -static bool isNurbselUV(const Nurb *nu, int flag, int *r_u, int *r_v) +static bool isNurbselUV(const Nurb *nu, uint8_t flag, int *r_u, int *r_v) { /* return (u != -1): 1 row in u-direction selected. U has value between 0-pntsv * return (v != -1): 1 column in v-direction selected. V has value between 0-pntsu @@ -1743,7 +1743,7 @@ static void rotateflagNurb(ListBase *editnurb, } } -void ed_editnurb_translate_flag(ListBase *editnurb, short flag, const float vec[3]) +void ed_editnurb_translate_flag(ListBase *editnurb, uint8_t flag, const float vec[3]) { /* all verts with ('flag' & flag) translate */ Nurb *nu; @@ -2052,7 +2052,7 @@ static void ed_curve_delete_selected(Object *obedit, View3D *v3d) } /* only for OB_SURF */ -bool ed_editnurb_extrude_flag(EditNurb *editnurb, const short flag) +bool ed_editnurb_extrude_flag(EditNurb *editnurb, const uint8_t flag) { Nurb *nu; BPoint *bp, *bpn, *newbp; @@ -2194,7 +2194,7 @@ static bool calc_duplicate_actvert( } static void adduplicateflagNurb( - Object *obedit, View3D *v3d, ListBase *newnurb, const short flag, const bool split) + Object *obedit, View3D *v3d, ListBase *newnurb, const uint8_t flag, const bool split) { ListBase *editnurb = object_editcurve_get(obedit); Nurb *nu, *newnu; diff --git a/source/blender/editors/curve/editcurve_query.c b/source/blender/editors/curve/editcurve_query.c index 774065b06ff..48571ab2a9b 100644 --- a/source/blender/editors/curve/editcurve_query.c +++ b/source/blender/editors/curve/editcurve_query.c @@ -62,7 +62,7 @@ static void ED_curve_pick_vert__do_closest(void *userData, bool is_changed; } *data = userData; - short flag; + uint8_t flag; float dist_test; if (bp) { diff --git a/source/blender/editors/curve/editcurve_select.c b/source/blender/editors/curve/editcurve_select.c index 4b502e9e336..aa4ba332b66 100644 --- a/source/blender/editors/curve/editcurve_select.c +++ b/source/blender/editors/curve/editcurve_select.c @@ -57,7 +57,7 @@ #include "DEG_depsgraph.h" /* returns 1 in case (de)selection was successful */ -bool select_beztriple(BezTriple *bezt, bool selstatus, short flag, eVisible_Types hidden) +bool select_beztriple(BezTriple *bezt, bool selstatus, uint8_t flag, eVisible_Types hidden) { if ((bezt->hide == 0) || (hidden == HIDDEN)) { if (selstatus == SELECT) { /* selects */ @@ -77,7 +77,7 @@ bool select_beztriple(BezTriple *bezt, bool selstatus, short flag, eVisible_Type } /* returns 1 in case (de)selection was successful */ -bool select_bpoint(BPoint *bp, bool selstatus, short flag, bool hidden) +bool select_bpoint(BPoint *bp, bool selstatus, uint8_t flag, bool hidden) { if ((bp->hide == 0) || (hidden == 1)) { if (selstatus == SELECT) { |