Manuscript (Case [Issue]21758) Number - Replace #min:max with #clampLow:High:

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Manuscript (Case [Issue]21758) Number - Replace #min:max with #clampLow:High:

Pharo Issue Tracker
Manuscript Notification
avatar
Enhancement in Project:  Number: 1. Pharo Image  •  You are subscribed to this case
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 Priority: 5 – Fix If Time Status Status: Work Needed
Assigned To Assigned to: Everyone Milestone Milestone: Pharo7.0

Go to Case
No longer need updates? Unsubscribe from this case.

Don't want Manuscript notifications anymore? Update your preferences.

Manuscript

_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
https://lists.gforge.inria.fr/mailman/listinfo/pharo-bugtracker