G'day,
I noticed that if you enter an invalid value for a PercentParameter an
IllegalArgumentException is raised, whereas I'd expect a warning message
to be displayed in the GUI as when invalid values are provided for other
types of parameter.
I traced the IllegalArgumentException to this line in
PercentParameter.setValueFromComponent()
if ((componentValue < 0) || (componentValue > 100))
throw new IllegalArgumentException("Invalid percentage value");
Should this bounds check instead be put into PercentParameter.checkValue()?
Additionally, should the check be softened such that percentages > 100%
are considered valid?
Or, perhaps 0% .. 100% might be the default range with configurable min
and max made possible as with other numeric parameters?
Please let me know your thoughts - then I'll open an item in the tracker.
Thanks,
Chris.
This message may contain confidential information. If you are not the
designated recipient, please notify the sender immediately, and delete the
original and any copies. Any use of the message by you is prohibited.
------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure
contains a definitive record of customers, application performance,
security threats, fraudulent activity, and more. Splunk takes this
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
_______________________________________________
Mzmine-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mzmine-devel
