[PR #58] [MERGED] Re-enable stream extraction for MScab #89

Closed
opened 2026-01-29 21:16:58 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/SabreTools/SabreTools.Serialization/pull/58
Author: @HeroponRikiBestest
Created: 1/25/2026
Status: Merged
Merged: 1/25/2026
Merged by: @mnadareski

Base: mainHead: mscab-stream


📝 Commits (3)

  • 75d9917 Re-enable extraction from stream
  • 0c19b6f Remove todo pre PR
  • 4115fd3 Properly support all situations, remove whitespace

📊 Changes

1 file changed (+23 additions, -17 deletions)

View changed files

📝 SabreTools.Serialization/Wrappers/MicrosoftCabinet.Extraction.cs (+23 -17)

📄 Description

Minimal work was required since most of the existing code already handled partial archives fine. Worked properly from my testing.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/SabreTools/SabreTools.Serialization/pull/58 **Author:** [@HeroponRikiBestest](https://github.com/HeroponRikiBestest) **Created:** 1/25/2026 **Status:** ✅ Merged **Merged:** 1/25/2026 **Merged by:** [@mnadareski](https://github.com/mnadareski) **Base:** `main` ← **Head:** `mscab-stream` --- ### 📝 Commits (3) - [`75d9917`](https://github.com/SabreTools/SabreTools.Serialization/commit/75d9917462a96c61d57ddebaab4418bba4c1bc51) Re-enable extraction from stream - [`0c19b6f`](https://github.com/SabreTools/SabreTools.Serialization/commit/0c19b6fcaf63fb3ad4e7160317a18f6d73cf4282) Remove todo pre PR - [`4115fd3`](https://github.com/SabreTools/SabreTools.Serialization/commit/4115fd3219627ba30315ebf3d0560149b84e8f78) Properly support all situations, remove whitespace ### 📊 Changes **1 file changed** (+23 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `SabreTools.Serialization/Wrappers/MicrosoftCabinet.Extraction.cs` (+23 -17) </details> ### 📄 Description Minimal work was required since most of the existing code already handled partial archives fine. Worked properly from my testing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 21:16:58 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SabreTools/SabreTools.Serialization#89