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

Dispose iterator after use #870

Merged
merged 4 commits into from
May 20, 2020

Conversation

earlbread
Copy link
Contributor

No description provided.

@earlbread earlbread self-assigned this May 19, 2020
moreal
moreal previously approved these changes May 19, 2020
@codecov
Copy link

codecov bot commented May 19, 2020

Codecov Report

Merging #870 into 0.9-maintenance will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@                 Coverage Diff                 @@
##           0.9-maintenance     #870      +/-   ##
===================================================
+ Coverage            87.55%   87.56%   +0.01%     
===================================================
  Files                  250      250              
  Lines                22675    22675              
===================================================
+ Hits                 19853    19856       +3     
+ Misses                1476     1473       -3     
  Partials              1346     1346              
Impacted Files Coverage Δ
Libplanet.RocksDBStore/RocksDBStore.cs 95.52% <100.00%> (ø)
Libplanet/Net/Protocols/KBucket.cs 85.59% <0.00%> (-2.55%) ⬇️
Libplanet/Net/Protocols/KademliaProtocol.cs 64.63% <0.00%> (-0.39%) ⬇️
Libplanet/Net/Swarm.cs 84.97% <0.00%> (+0.51%) ⬆️

riemannulus
riemannulus previously approved these changes May 19, 2020
Copy link
Contributor

@dahlia dahlia left a comment

Choose a reason for hiding this comment

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

  • How about merging this change to 0.9-maintenance instead of master?
  • I believe this change can be recorded in the changelog, because this fixes a sort of memory leak.

longfin
longfin previously approved these changes May 19, 2020
@earlbread earlbread dismissed stale reviews from longfin, riemannulus, and moreal via 8de6b07 May 20, 2020 01:14
@earlbread earlbread changed the base branch from master to 0.9-maintenance May 20, 2020 01:14
@earlbread
Copy link
Contributor Author

I changed the base to 0.9-maintenance and added a changelog.

longfin
longfin previously approved these changes May 20, 2020
moreal
moreal previously approved these changes May 20, 2020
riemannulus
riemannulus previously approved these changes May 20, 2020
CHANGES.md Outdated Show resolved Hide resolved
Co-authored-by: Hong Minhee <hong.minhee@gmail.com>
@earlbread earlbread dismissed stale reviews from riemannulus and moreal via fc6a98f May 20, 2020 04:29
@earlbread earlbread merged commit c08e9e1 into planetarium:0.9-maintenance May 20, 2020
@earlbread earlbread deleted the fix-iterator-dispose branch May 20, 2020 05:04
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.

5 participants