Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix TreeSet.ofAll with comparator and empty values #2255

Merged
merged 4 commits into from
Jun 18, 2018

Conversation

nfekete
Copy link
Member

@nfekete nfekete commented Jun 14, 2018

fixes #2254

@nfekete nfekete changed the title fix for #2254 fix TreeSet.ofAll with comparator and empty values Jun 14, 2018
@codecov-io
Copy link

codecov-io commented Jun 14, 2018

Codecov Report

Merging #2255 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2255   +/-   ##
=========================================
  Coverage     97.17%   97.17%           
  Complexity     5344     5344           
=========================================
  Files            94       94           
  Lines         12288    12288           
  Branches       1596     1596           
=========================================
  Hits          11941    11941           
  Misses          193      193           
  Partials        154      154
Impacted Files Coverage Δ Complexity Δ
vavr/src/main/java/io/vavr/collection/TreeSet.java 100% <100%> (ø) 169 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a9b7b0e...013b55a. Read the comment docs.

@ruslansennov ruslansennov merged commit 5e8acec into vavr-io:master Jun 18, 2018
@ruslansennov
Copy link
Member

thank you

@ruslansennov ruslansennov added this to the vavr-1.0.0 milestone Jun 18, 2018
@nfekete nfekete deleted the issue-2254 branch June 18, 2018 13:06
CauchyPeano pushed a commit to CauchyPeano/vavr that referenced this pull request Jul 24, 2018
* test for TreeSet.removeAll case that removes all elements from the set

* fix TreeSet.ofAll() discarding the comparator when creating empty set

* removed unnecessary @SuppressWarnings

* test for TreeSet.ofAll with explicit comparator and empty initial values
danieldietrich pushed a commit that referenced this pull request Jan 6, 2019
* test for TreeSet.removeAll case that removes all elements from the set

* fix TreeSet.ofAll() discarding the comparator when creating empty set

* removed unnecessary @SuppressWarnings

* test for TreeSet.ofAll with explicit comparator and empty initial values
@danieldietrich danieldietrich modified the milestones: v1.0.0, v0.9.3 Jan 7, 2019
danieldietrich pushed a commit that referenced this pull request Jan 16, 2019
* test for TreeSet.removeAll case that removes all elements from the set

* fix TreeSet.ofAll() discarding the comparator when creating empty set

* removed unnecessary @SuppressWarnings

* test for TreeSet.ofAll with explicit comparator and empty initial values
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ClassCastException during addAll to TreeSet
4 participants