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

Implement ImmutableHashSet<T> #50

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

sharwell
Copy link
Member

Fixes #39

@sharwell sharwell self-assigned this Mar 27, 2017
@sharwell sharwell changed the title Immutable hash set Implement ImmutableHashSet<T> Mar 27, 2017
@codecov
Copy link

codecov bot commented Mar 27, 2017

Codecov Report

Merging #50 into master will increase coverage by 0.79%.
The diff coverage is 88.37%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #50      +/-   ##
============================================
+ Coverage     81.17%   81.97%   +0.79%     
- Complexity     1167     1325     +158     
============================================
  Files            40       44       +4     
  Lines          6938     7693     +755     
  Branches        877      965      +88     
============================================
+ Hits           5632     6306     +674     
- Misses         1011     1073      +62     
- Partials        295      314      +19
Impacted Files Coverage Δ
src/com/tvl/util/StrongBox.java 100% <100%> (ø)
test/com/tvl/util/ImmutableSetTest.java 98.4% <100%> (+1.27%) ⬆️
src/com/tvl/util/Immutables.java 83.33% <57.14%> (-10.79%) ⬇️
test/com/tvl/util/ImmutableHashSetBuilderTest.java 87.57% <87.57%> (ø)
src/com/tvl/util/ImmutableHashSet.java 88.05% <88.05%> (ø)
test/com/tvl/util/ImmutableHashSetTest.java 93.25% <93.25%> (ø)
... and 4 more

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 0e988db...250c1d3. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant