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