microsoft / microsoft/RecursiveExtractor
Convert FileEntry To IDisposable
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- C#
- Stars
- 220
- Forks
- 35
- Avg merge
- 20d 1h
- Merged PRs (30d)
- 1
Description
We should investigate making FileEntry inherit IDisposable instead of relying on the finalizer to clean up the Content stream.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the FileEntry class and trace how its Content stream and finalizer are used by callers. Determine the disposal lifecycle and update the behavior so resource cleanup is explicit through IDisposable rather than relying on finalization; existing tests should continue to pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100