Re: [Mzmine-devel] Problem with PercentParameter

2011-12-02 Thread Tomas Pluskal
Hi Chris, You are correct, the check should be done in the checkValue() function. I think the code remained there back from the days when there was no checkValue() function. I committed a fix now. I also added the configurable range (0-100% is default). Cheers, Tomas On Dec 2, 2011, at 12:1

[Mzmine-devel] Problem with PercentParameter

2011-12-01 Thread Pudney Chris (ext) GBJH
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