Fix overread warning in screen operator
Fixes a `stringop-overread` warning, other people are working on fixing some more :)
This commit is contained in:
parent
60bd6804db
commit
5d74fa314b
|
@ -2608,7 +2608,7 @@ typedef struct RegionMoveData {
|
|||
ARegion *region;
|
||||
ScrArea *area;
|
||||
int bigger, smaller, origval;
|
||||
int origx, origy;
|
||||
int orig_xy[2];
|
||||
int maxsize;
|
||||
AZEdge edge;
|
||||
|
||||
|
@ -2716,8 +2716,7 @@ static int region_scale_invoke(bContext *C, wmOperator *op, const wmEvent *event
|
|||
}
|
||||
rmd->area = sad->sa1;
|
||||
rmd->edge = az->edge;
|
||||
rmd->origx = event->xy[0];
|
||||
rmd->origy = event->xy[1];
|
||||
copy_v2_v2_int(rmd->orig_xy, event->xy);
|
||||
rmd->maxsize = area_max_regionsize(rmd->area, rmd->region, rmd->edge);
|
||||
|
||||
/* if not set we do now, otherwise it uses type */
|
||||
|
@ -2804,7 +2803,7 @@ static int region_scale_modal(bContext *C, wmOperator *op, const wmEvent *event)
|
|||
(BLI_rcti_size_x(&rmd->region->v2d.mask) + 1);
|
||||
const int snap_size_threshold = (U.widget_unit * 2) / aspect;
|
||||
if (ELEM(rmd->edge, AE_LEFT_TO_TOPRIGHT, AE_RIGHT_TO_TOPLEFT)) {
|
||||
delta = event->xy[0] - rmd->origx;
|
||||
delta = event->xy[0] - rmd->orig_xy[0];
|
||||
if (rmd->edge == AE_LEFT_TO_TOPRIGHT) {
|
||||
delta = -delta;
|
||||
}
|
||||
|
@ -2837,7 +2836,7 @@ static int region_scale_modal(bContext *C, wmOperator *op, const wmEvent *event)
|
|||
}
|
||||
}
|
||||
else {
|
||||
delta = event->xy[1] - rmd->origy;
|
||||
delta = event->xy[1] - rmd->orig_xy[1];
|
||||
if (rmd->edge == AE_BOTTOM_TO_TOPLEFT) {
|
||||
delta = -delta;
|
||||
}
|
||||
|
@ -2879,7 +2878,7 @@ static int region_scale_modal(bContext *C, wmOperator *op, const wmEvent *event)
|
|||
}
|
||||
case LEFTMOUSE:
|
||||
if (event->val == KM_RELEASE) {
|
||||
if (len_manhattan_v2v2_int(event->xy, &rmd->origx) <= WM_EVENT_CURSOR_MOTION_THRESHOLD) {
|
||||
if (len_manhattan_v2v2_int(event->xy, rmd->orig_xy) <= WM_EVENT_CURSOR_MOTION_THRESHOLD) {
|
||||
if (rmd->region->flag & RGN_FLAG_HIDDEN) {
|
||||
region_scale_toggle_hidden(C, rmd);
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue