whether&when to dispose OutputStream #88

Open
opened 2026-01-29 22:06:25 +00:00 by claunia · 0 comments
Owner

Originally created by @Supegg on GitHub (Mar 3, 2016).

i think the caller decides whether&when to dispose outputstream.
so, i suggest
public TarWriter(Stream destination, CompressionInfo compressionInfo, bool closeStream = true)
: base(ArchiveType.Tar)
public GZipWriter(Stream destination, bool closeStream = true)
: base(ArchiveType.GZip)
public ZipWriter(Stream destination, CompressionInfo compressionInfo, string zipComment, bool closeStream = true)
: base(ArchiveType.Zip)

PS: why? OutputStream.Dispose(); // required when bzip2 compression is used

Originally created by @Supegg on GitHub (Mar 3, 2016). i think the caller decides whether&when to dispose outputstream. so, i suggest public TarWriter(Stream destination, CompressionInfo compressionInfo, bool closeStream = true) : base(ArchiveType.Tar) public GZipWriter(Stream destination, bool closeStream = true) : base(ArchiveType.GZip) public ZipWriter(Stream destination, CompressionInfo compressionInfo, string zipComment, bool closeStream = true) : base(ArchiveType.Zip) PS: why? OutputStream.Dispose(); // required when bzip2 compression is used
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/sharpcompress#88