microsoft / microsoft/RecursiveExtractor
Harmonize Using/PassThroughStream Arguments in Extractors
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 220
- Forks
- 35
- Avg merge
- 20d 1h
- Merged PRs (30d)
- 1
Description
Some Extractors use the using pattern on StreamFactory.GetAppropriateBackingStream and some use passthrough stream on a created FileEntry and rely on the finalizer to clean up the stream. We should harmonize all extractors on a single scheme for this wherever possible.
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 extractors that call StreamFactory.GetAppropriateBackingStream with using and those that pass through a stream on a created FileEntry. Compare their cleanup and finalizer behavior, then determine a single ownership scheme that can be applied wherever possible. Done means the extractors are harmonized without relying on inconsistent stream cleanup.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100