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