diff options
author | Campbell Barton <ideasman42@gmail.com> | 2014-02-03 11:59:36 +0400 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2014-02-03 12:35:44 +0400 |
commit | 107566ebf1cf7d563a00f64615410c2b7d443c00 (patch) | |
tree | d10365de620ea16485b82034ee13df2231a5ea4c /source/blender/editors/screen/screen_ops.c | |
parent | d900f5be55d0812eb5c7dfd3ea47020c3edafd41 (diff) |
Code cleanup: check -1 for error return explicitly
Diffstat (limited to 'source/blender/editors/screen/screen_ops.c')
-rw-r--r-- | source/blender/editors/screen/screen_ops.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/source/blender/editors/screen/screen_ops.c b/source/blender/editors/screen/screen_ops.c index ae1b7cc937f..53a1f4fc8ca 100644 --- a/source/blender/editors/screen/screen_ops.c +++ b/source/blender/editors/screen/screen_ops.c @@ -2516,7 +2516,7 @@ static int area_join_modal(bContext *C, wmOperator *op, const wmEvent *event) if (sa) { if (jd->sa1 != sa) { dir = area_getorientation(jd->sa1, sa); - if (dir >= 0) { + if (dir != -1) { if (jd->sa2) jd->sa2->flag &= ~AREA_FLAG_DRAWJOINTO; jd->sa2 = sa; jd->sa2->flag |= AREA_FLAG_DRAWJOINTO; @@ -2527,7 +2527,7 @@ static int area_join_modal(bContext *C, wmOperator *op, const wmEvent *event) * in this case we can swap areas. */ dir = area_getorientation(sa, jd->sa2); - if (dir >= 0) { + if (dir != -1) { if (jd->sa1) jd->sa1->flag &= ~AREA_FLAG_DRAWJOINFROM; if (jd->sa2) jd->sa2->flag &= ~AREA_FLAG_DRAWJOINTO; jd->sa1 = jd->sa2; @@ -2553,13 +2553,13 @@ static int area_join_modal(bContext *C, wmOperator *op, const wmEvent *event) if (jd->sa1) jd->sa1->flag |= AREA_FLAG_DRAWJOINFROM; if (jd->sa2) jd->sa2->flag |= AREA_FLAG_DRAWJOINTO; dir = area_getorientation(jd->sa1, jd->sa2); - if (dir < 0) { + if (dir == -1) { printf("oops, didn't expect that!\n"); } } else { dir = area_getorientation(jd->sa1, sa); - if (dir >= 0) { + if (dir != -1) { if (jd->sa2) jd->sa2->flag &= ~AREA_FLAG_DRAWJOINTO; jd->sa2 = sa; jd->sa2->flag |= AREA_FLAG_DRAWJOINTO; |