fix for MeasurementScheduleManagerBean.update

Ian Springer ian.springer at redhat.com
Tue Nov 2 14:14:45 UTC 2010


  Yeah, I don't like it either. I was just trying to avoid API changes. 
But since that method is only in the local interface, I guess I can add 
a new method, or methods, with less cryptic semantics, and then 
deprecate, or even just remove, this one.

On 11/02/2010 10:07 AM, Heiko W.Rupp wrote:
> 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
>


-- 
Ian Springer
Principal Software Developer
JBoss Operations Network
Red Hat
ian.springer at redhat.com



More information about the rhq-devel mailing list