From 2a73e463151f4105fe60730c9163af305c30ca87 Mon Sep 17 00:00:00 2001 From: Přemysl Eric Janouch Date: Sat, 19 Feb 2022 23:03:49 +0100 Subject: fiv-jpegcrop: avoid negatively sized crop regions --- fiv-jpegcrop.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/fiv-jpegcrop.c b/fiv-jpegcrop.c index 4cf9d64..4bbcd97 100644 --- a/fiv-jpegcrop.c +++ b/fiv-jpegcrop.c @@ -214,13 +214,14 @@ on_mouse(guint state, guint button, gdouble x, gdouble y) switch (button) { case GDK_BUTTON_PRIMARY: - g.left = MAX(0, (int) (x - 1)) / g.mcu_width * g.mcu_width; - g.top = MAX(0, (int) (y - 1)) / g.mcu_height * g.mcu_height; + g.left = CLAMP((int) (x - 1), 0, g.right) / g.mcu_width * g.mcu_width; + g.top = CLAMP((int) (y - 1), 0, g.bottom) / g.mcu_height * g.mcu_height; update(); return TRUE; case GDK_BUTTON_SECONDARY: - g.right = MIN(x, g.width); // Inclusive of pointer position. - g.bottom = MIN(y, g.height); // Inclusive of pointer position. + // Inclusive of pointer position. + g.right = CLAMP(x, g.left, g.width); + g.bottom = CLAMP(y, g.top, g.height); update(); return TRUE; default: -- cgit v1.2.3-70-g09d2