diff options
| author | Přemysl Eric Janouch <p@janouch.name> | 2022-02-19 23:03:49 +0100 |
|---|---|---|
| committer | Přemysl Eric Janouch <p@janouch.name> | 2022-02-19 23:04:12 +0100 |
| commit | 2a73e463151f4105fe60730c9163af305c30ca87 (patch) | |
| tree | 847687c7208fb1a90b60d651c67b5609685c1259 | |
| parent | a68a73cf5c3cf652c1d30c5948cb7040e60d5f5e (diff) | |
| download | fiv-2a73e463151f4105fe60730c9163af305c30ca87.tar.gz fiv-2a73e463151f4105fe60730c9163af305c30ca87.tar.xz fiv-2a73e463151f4105fe60730c9163af305c30ca87.zip | |
fiv-jpegcrop: avoid negatively sized crop regions
| -rw-r--r-- | fiv-jpegcrop.c | 9 |
1 files 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: |