Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Provide Seq(..).maxOption and friends#6574

Merged
szeiger merged 1 commit into
scala:2.13.xfrom
stephennancekivell:safe-min-max
May 2, 2018
Merged

Provide Seq(..).maxOption and friends#6574
szeiger merged 1 commit into
scala:2.13.xfrom
stephennancekivell:safe-min-max

Conversation

@stephennancekivell
Copy link
Copy Markdown
Contributor

@stephennancekivell stephennancekivell commented Apr 28, 2018

Provide Seq(..).maxOption and friends so developers can easily write code that handles the empty case.

@scala-jenkins scala-jenkins added this to the 2.13.0-M4 milestone Apr 28, 2018
@stephennancekivell
Copy link
Copy Markdown
Contributor Author

stephennancekivell commented Apr 28, 2018

Just signed the cla. Please validate again. :)

@SethTisue
Copy link
Copy Markdown
Member

SethTisue commented Apr 28, 2018 

[test] [error] /home/jenkins/workspace/scala-2.13.x-validate-main@2/test/junit/scala/collection/TraversableOnceTest.scala:81: diverging implicit expansion for type scala.math.Ordering[T1]
[test] [error] starting with object Int in object Ordering
[test] [error]     assert(Nil.maxOption == None, "maxOption on a Empty Iterable is None")
[test] [error]                ^
[test] [error] /home/jenkins/workspace/scala-2.13.x-validate-main@2/test/junit/scala/collection/TraversableOnceTest.scala:82: diverging implicit expansion for type scala.math.Ordering[T1]
[test] [error] starting with object Int in object Ordering
[test] [error]     assert(Nil.minOption == None, "minOption on a Empty Iterable is None")
[test] [error]                ^
[test] [error] /home/jenkins/workspace/scala-2.13.x-validate-main@2/test/junit/scala/collection/TraversableOnceTest.scala:83: diverging implicit expansion for type scala.math.Ordering[T1]
[test] [error] starting with object Int in object Ordering
[test] [error]     assert(Nil.maxByOption(identity) == None, "maxByOption on a Empty Iterable is None")
[test] [error]                           ^
[test] [error] /home/jenkins/workspace/scala-2.13.x-validate-main@2/test/junit/scala/collection/TraversableOnceTest.scala:84: diverging implicit expansion for type scala.math.Ordering[T1]
[test] [error] starting with object Int in object Ordering
[test] [error]     assert(Nil.minByOption(identity) == None, "minByOption on a Empty Iterable is None")
[test] [error]                           ^
[test] [error] four errors found

@stephennancekivell
Copy link
Copy Markdown
Contributor Author

stephennancekivell commented Apr 28, 2018

Oh my, thanks Seth. Funny I dont get that when I run from idea. Anyway fixed and updated.

martin-g
martin-g reviewed Apr 28, 2018
View reviewed changes
Comment thread test/junit/scala/collection/TraversableOnceTest.scala Outdated

@Test
def checkNonEmptyOption: Unit = {
assert(Seq(1).maxOption == Some(1), "maxOption on a Empty Iterable has option")
Copy link
Copy Markdown

@martin-g martin-g Apr 28, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on a Empty Iterable ?

Copy link
Copy Markdown
Contributor Author

@stephennancekivell stephennancekivell Apr 28, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opps, sorry thanks, fixed.

@stephennancekivell stephennancekivell changed the title provide minOption and maxOption Provide Seq(..).maxOption and friends Apr 29, 2018
szeiger
szeiger approved these changes May 2, 2018
View reviewed changes
Copy link
Copy Markdown
Contributor

@szeiger szeiger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a useful addition to me.

@szeiger szeiger merged commit 698ec6b into scala:2.13.x May 2, 2018
@SethTisue
Copy link
Copy Markdown
Member

SethTisue commented May 2, 2018

thanks @stephennancekivell! this is now your little patch of land in the Scala standard library :-)

@stephennancekivell stephennancekivell deleted the safe-min-max branch May 3, 2018 11:29
@stephennancekivell
Copy link
Copy Markdown
Contributor Author

stephennancekivell commented May 3, 2018

haha, thanks @SethTisue

@SethTisue SethTisue added the release-notes worth highlighting in next release notes label May 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SethTisue SethTisue SethTisue approved these changes

+2 more reviewers

@martin-g martin-g martin-g left review comments

@szeiger szeiger szeiger approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

release-notes worth highlighting in next release notes

Projects

None yet

Milestone

2.13.0-M4

Development

Successfully merging this pull request may close these issues.

5 participants

@stephennancekivell @SethTisue @szeiger @martin-g @scala-jenkins