[swift-evolution] [Proposal] Add floor() and ceiling() functions to FloatingPoint

Karl Wagner razielim at gmail.com
Wed Jun 29 19:41:49 CDT 2016


OK I’ve incorporated those small things and submitted it for review. We’ve had an initial discussion, incorporated all the feedback, so I think it’s met the bar for a PR 👍

https://github.com/apple/swift-evolution/pull/394

Karl

> On 29 Jun 2016, at 20:20, Xiaodi Wu <xiaodi.wu at gmail.com> wrote:
> 
> On Wed, Jun 29, 2016 at 1:07 PM, Stephen Canon <scanon at apple.com <mailto:scanon at apple.com>> wrote:
> 
>> On Jun 29, 2016, at 2:06 PM, Xiaodi Wu <xiaodi.wu at gmail.com <mailto:xiaodi.wu at gmail.com>> wrote:
>> 
>> On Wed, Jun 29, 2016 at 12:12 PM, Stephen Canon via swift-evolution <swift-evolution at swift.org <mailto:swift-evolution at swift.org>> wrote:
>> 
>>> On Jun 29, 2016, at 1:10 PM, Matthew Johnson <matthew at anandabits.com <mailto:matthew at anandabits.com>> wrote:
>>> 
>>> My criticism of the 'toNearestOrGreatest' still stands though.  I think this name is misleading given the stated semantics.  The name indicates "greater value" not "greater magnitude" which are opposites in the case of negative numbers.
>> 
>> Yup, I agree.  I think I originally suggested `toNearestTiesAway`.  I’m not tied to that name specifically, but we should be clear that ties go away from zero, not up.
>> 
>> Agreed; `toNearestOrAwayFromZero` is the most accurate and consistent description.
> 
> Worth noting that since this is the defaulted behavior, we can get away with a wordy description.
> 
> This is really picking nits, but the most common behavior would be best as the first enum case, no? Maybe go roughly from most-value-preserving to least-value-preserving, like so:
> 
> `enum RoundingRule { case toNearestOrAwayFromZero, toNearestOrEven, up, down, towardZero }`
> 
>  
> 
> – Steve

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-evolution/attachments/20160630/c1c54156/attachment.html>


More information about the swift-evolution mailing list