search

pjtatlow / jammdb

Just Another Memory Mapped Database
Apache License 2.0
269 stars 20 forks source link

add lifetimes to cursors #4

Closed pjtatlow closed 4 years ago

pjtatlow commented 4 years ago

This PR adds lifetimes to the Cursor struct which makes it so you can't use it unsafely after the transaction is dropped.

Closes #3

codecov[bot] commented 4 years ago

Codecov Report

Merging #4 into master will decrease coverage by 0.02%. The diff coverage is 92.85%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   93.10%   93.07%   -0.03%     
==========================================
  Files          18       18              
  Lines        1783     1790       +7     
==========================================
+ Hits         1660     1666       +6     
- Misses        123      124       +1
Impacted Files Coverage Δ
src/cursor.rs 84.25% <50.00%> (-0.79%) :arrow_down:
src/bucket.rs 92.88% <100.00%> (+0.15%) :arrow_up:

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 f0db8ca...763ea07. Read the comment docs.