Skip to content

Conversation

@kernelmethod
Copy link
Owner

Add SkiplistSet, which is identical to Skiplist but only permits one copy of each key.

…ich will be used to indicate whether the Skiplist is a list or a set.
- Ensure that sentinel nodes are marked as fully linked.
- Stop using vec() to convert a Skiplist into a Vector. From now on,
  collect() should be used (in line with other iterators in Julia).
- Add an implementation of Base.insert! for SkiplistSet.
@codecov-commenter
Copy link

Codecov Report

Merging #5 into master will decrease coverage by 3.37%.
The diff coverage is 89.47%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
- Coverage   91.04%   87.67%   -3.38%     
==========================================
  Files           4        4              
  Lines         134      146      +12     
==========================================
+ Hits          122      128       +6     
- Misses         12       18       +6     
Impacted Files Coverage Δ
src/Skiplists.jl 100.00% <ø> (ø)
src/core.jl 100.00% <ø> (ø)
src/node.jl 79.59% <73.33%> (-3.75%) ⬇️
src/list.jl 91.11% <95.23%> (-3.01%) ⬇️

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 5287d33...14ac42a. Read the comment docs.

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@5287d33). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   87.67%           
=========================================
  Files             ?        4           
  Lines             ?      146           
  Branches          ?        0           
=========================================
  Hits              ?      128           
  Misses            ?       18           
  Partials          ?        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 5287d33...14ac42a. Read the comment docs.

@kernelmethod kernelmethod merged commit 991a474 into master Jun 6, 2020
@kernelmethod
Copy link
Owner Author

Fixed #1

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.

3 participants