Skip to content

Add type to iterate directory#229

Merged
AkihiroSuda merged 1 commit intocontainerd:mainfrom
cpuguy83:dir_iter
Oct 18, 2023
Merged

Add type to iterate directory#229
AkihiroSuda merged 1 commit intocontainerd:mainfrom
cpuguy83:dir_iter

Conversation

@cpuguy83
Copy link
Copy Markdown
Member

@cpuguy83 cpuguy83 commented Aug 9, 2023

This prevents reading from directory from allocating an unbounded slice at the cost of potentially having to read more than once.

It also prevents unccessary allocation in cases where an error occurs handling the file type.

@cpuguy83 cpuguy83 force-pushed the dir_iter branch 4 times, most recently from e0a78ae to 67e1035 Compare August 9, 2023 23:00
This prevents reading from directory from allocating an unbounded slice
at the cost of potentially having to read more than once.

It also prevents unccessary allocation in cases where an error occurs
handling the file type.

Signed-off-by: Brian Goff <[email protected]>
@AkihiroSuda AkihiroSuda requested a review from samuelkarp August 20, 2023 00:08
@AkihiroSuda
Copy link
Copy Markdown
Member

Let's make a new release after merging this

@dmcgowan dmcgowan changed the title Add type to itterate directory Add type to iterate directory Oct 18, 2023
@AkihiroSuda AkihiroSuda merged commit 1743d91 into containerd:main Oct 18, 2023
@AkihiroSuda
Copy link
Copy Markdown
Member

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.

4 participants