zynaddsubfx

ZynAddSubFX open source synthesizer
Log | Files | Refs | Submodules | LICENSE

commit 8e868b3e0a93d5de5605ba40ad4c9e5d424298d9
parent 1865bfaa70ba82fc88da06731cc93cd55b72e190
Author: Christopher A. Oliver <[email protected]>
Date:   Sat, 17 Oct 2015 03:33:00 -0400

Clamped an unlimited arg to value.  Use clamp rather than limit for value.
It already knows about min and max.

Diffstat:
Msrc/UI/Fl_Osc_Slider.cpp | 4++--
Msrc/UI/WidgetPDial.cpp | 6+++---
2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/UI/Fl_Osc_Slider.cpp b/src/UI/Fl_Osc_Slider.cpp @@ -112,7 +112,7 @@ int Fl_Osc_Slider::handle(int ev, int X, int Y, int W, int H) return 1; case FL_RELEASE: rounded = value() + 0.5; - value(limit((double)rounded,minimum(),maximum())); + value(clamp((double)rounded)); if (Fl::event_clicks() == 1) { Fl::event_clicks(0); reset_requested = true; @@ -138,7 +138,7 @@ int Fl_Osc_Slider::handle(int ev, int X, int Y, int W, int H) handled = Fl_Slider::handle(ev, X, Y, W, H); int delta = value() - old_value; if (ev == FL_DRAG && (delta < -1 || delta > 1)) { - value(old_value + (delta > 0 ? 1 : -1)); + value(clamp((old_value + (delta > 0 ? 1 : -1)))); value_damage(); do_callback(); } diff --git a/src/UI/WidgetPDial.cpp b/src/UI/WidgetPDial.cpp @@ -66,7 +66,7 @@ int WidgetPDial::handle(int event) else dragsize = Fl::event_button1() ? 200.0f : 2000.0f; - value(limit(oldvalue + dy / dragsize * (max - min), min, max)); + value(clamp(oldvalue + dy / dragsize * (max - min))); tipwin->showValue(transform(value())); value_damage(); if(this->when() != 0) @@ -94,7 +94,7 @@ int WidgetPDial::handle(int event) } result = value() + dy / dragsize * (max - min); } - value(limit(result, min, max)); + value(clamp(result)); tipwin->showValue(transform(value())); value_damage(); @@ -113,7 +113,7 @@ int WidgetPDial::handle(int event) case FL_RELEASE: if (integer_step) { int rounded = value() + 0.5; - value(limit((double)rounded,min,max)); + value(clamp(rounded)); } tipwin->hide(); resetPos();