fix for MeasurementScheduleManagerBean.update
Heiko W.Rupp
hrupp at redhat.com
Tue Nov 2 14:07:13 UTC 2010
Hey,
Am 02.11.2010 um 14:59 schrieb Ian Springer:
>> - boolean enableDisable = (collectionInterval> 0);
>> + boolean enable = (collectionInterval>= 0);
I am not sure I like this overloading too much, even if it makes sense.
A more explicit (= additional parameter) could be better.
Heiko
--
Reg. Adresse: Red Hat GmbH, Otto-Hahn-Strasse 20, 85609 Dornach bei München
Handelsregister: Amtsgericht München HRB 153243
Geschaeftsführer: Brendan Lane, Charlie Peters, Michael Cunningham, Charles Cachera
More information about the rhq-devel
mailing list