|
I'm not trying to diverge, you are trying the converge. :)
Neverthless, here's another option:
add #clampLow:high: method
change #min:max: to
min: aMin max: aMax ^ self clampLow: aMin max: aMax
that way
* people looking for the method can still find it (I don't think anyone looking for the current behavior would think about voltages, but about mins and maxes (and I suppose the non-discoverability applies to #beBetween:and: too) * when you look at the implementation, it is immediately obvious what it does
|
|
|
Priority: 5 – Fix If Time
|
|
Status: Work Needed
|
|
Assigned to: Everyone
|
|
Milestone: Pharo7.0
|
Go to Case
|
|