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

BUG: use size_t for holding the size of a directory, continue calculating dir size after we've hit file number limit #136

Merged
merged 1 commit into from
May 17, 2024

Conversation

ciscon
Copy link
Collaborator

@ciscon ciscon commented May 10, 2024

…rflow on larger directories and never clean things up. this is related to issue #135

@ciscon ciscon requested review from qqshka and tcsabina May 10, 2024 18:02
@ciscon ciscon force-pushed the use_larger_integer_for_size branch 2 times, most recently from 505d616 to a6a4ca2 Compare May 10, 2024 19:11
@ciscon ciscon changed the title BUG: use uint64 for holding the size of a directory, otherwise we ove… BUG: use size_t for holding the size of a directory, continue calculating dir size after we've hit file number limit May 10, 2024
…rflow on larger directories and never clean things up. also, continue iterating through files once we've hit the limit, we will stop appending new values to the files array but continue calculating directory size. this is related to issue #135
@ciscon ciscon force-pushed the use_larger_integer_for_size branch from a6a4ca2 to cf48f59 Compare May 10, 2024 19:27
@ciscon ciscon merged commit 4bdd739 into master May 17, 2024
18 checks passed
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