diff options
author | Daniel Dunbar <daniel@zuster.org> | 2005-07-15 02:16:38 +0400 |
---|---|---|
committer | Daniel Dunbar <daniel@zuster.org> | 2005-07-15 02:16:38 +0400 |
commit | 480b8ce10b149aa7aa9c62fed62cbf2a5ec904b3 (patch) | |
tree | 6762786764a5d29103df5925f9fae99446d457ef /source/blender/src/editview.c | |
parent | 40bfa5c7a1a04f2dad0825e2c93edd25b586cbd2 (diff) |
- switch em_{vert,solid,wire}offs to be unsigned
- some other twiddles to fix mixed signedness compiler warnings
And from the "Dear lazy programmers" file: After you have extern declared
a variable 5 or more times at the top a function, you would have actually
saved a lot of effort by just putting it in the header where it belonged.
Or perhaps you thought by hiding it no one else would notice you were
using global variables to pass information around. Tsk tsk.
Diffstat (limited to 'source/blender/src/editview.c')
-rw-r--r-- | source/blender/src/editview.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/source/blender/src/editview.c b/source/blender/src/editview.c index 9e6e4581644..66f0543d460 100644 --- a/source/blender/src/editview.c +++ b/source/blender/src/editview.c @@ -267,13 +267,13 @@ static void lasso_select_boundbox(rcti *rect, short mcords[][2], short moves) static void do_lasso_select_mesh(short mcords[][2], short moves, short select) { - extern int em_solidoffs, em_wireoffs; // let linker solve it... from editmesh_mods.c EditMesh *em = G.editMesh; EditVert *eve; EditEdge *eed; EditFace *efa; rcti rect; - int index, bbsel=0; // bbsel: no clip needed with screencoords + unsigned int index; + int bbsel=0; // bbsel: no clip needed with screencoords lasso_select_boundbox(&rect, mcords, moves); @@ -434,7 +434,6 @@ static void do_lasso_select_lattice(short mcords[][2], short moves, short select static void do_lasso_select_facemode(short mcords[][2], short moves, short select) { - extern int em_vertoffs; // still bad code, let linker solve for now Mesh *me; TFace *tface; rcti rect; @@ -1274,7 +1273,6 @@ void borderselect(void) else if(G.obedit) { /* used to be a bigger test, also included sector and life */ if(G.obedit->type==OB_MESH) { - extern int em_solidoffs, em_wireoffs; // let linker solve it... from editmesh_mods.c EditMesh *em = G.editMesh; EditVert *eve; EditEdge *eed; @@ -1573,7 +1571,6 @@ void borderselect(void) static void mesh_selectionCB(int selecting, Object *editobj, short *mval, float rad) { - extern int em_solidoffs, em_wireoffs; // let linker solve it... from editmesh_mods.c EditMesh *em = G.editMesh; EditVert *eve; EditEdge *eed; |