Skip to content

Change ZipBlocks structs to classes #113453

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
carlossanlop merged 11 commits into from
Mar 21, 2025
Merged

Change ZipBlocks structs to classes #113453

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
994e451
Autogenerated and unavoidable sln changes.
carlossanlop Mar 12, 2025
60b94ac
Change ZipBlocks structs to classes. Update the places where they're …
carlossanlop Mar 12, 2025
452e3eb
Revert "Autogenerated and unavoidable sln changes."
carlossanlop Mar 14, 2025
8ca5770
_data ??= [] as a single expression
carlossanlop Mar 14, 2025
ea9827e
Update nullability usage around ZipCentralDirectoryFileHeader.TryRead…
carlossanlop Mar 20, 2025
44af7b8
Adjust nullability for zip64ExtraField.
carlossanlop Mar 20, 2025
0cb27fc
revert ZipLocalFileHeader type change, it can stay readonly partial s…
carlossanlop Mar 20, 2025
53515fb
Fix null propagation error, but add a comment so it's not easily igno…
carlossanlop Mar 20, 2025
f11f023
Merge branch 'ZipStructToClass' of https://github.com/carlossanlop/ru…
carlossanlop Mar 20, 2025
da47640
Simpler if condition.
carlossanlop Mar 20, 2025
b263a5a
Improve nullability in the rest of the Zip64ExtraField usages.
carlossanlop Mar 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/libraries/System.IO.Compression/src/System/IO/Compression/ZipArchive.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -530,7 +530,7 @@ private void ReadCentralDirectory()
while (continueReadingCentralDirectory
&& currPosition + ZipCentralDirectoryFileHeader.BlockConstantSectionSize < sizedFileBuffer.Length)
{
ZipCentralDirectoryFileHeader currentHeader = default;
ZipCentralDirectoryFileHeader currentHeader = new();
Comment thread
carlossanlop marked this conversation as resolved.
Outdated

continueReadingCentralDirectory = continueReadingCentralDirectory &&
ZipCentralDirectoryFileHeader.TryReadBlock(sizedFileBuffer.Slice(currPosition), _archiveStream,
Expand Down
4 changes: 2 additions & 2 deletions src/libraries/System.IO.Compression/src/System/IO/Compression/ZipArchiveEntry.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -491,7 +491,7 @@ internal void WriteCentralDirectoryFileHeader(bool forceWrite)
Debug.Assert(_fileComment.Length <= ushort.MaxValue);

// decide if we need the Zip64 extra field:
Zip64ExtraField zip64ExtraField = default;
Zip64ExtraField zip64ExtraField = new();
uint compressedSizeTruncated, uncompressedSizeTruncated, offsetOfLocalHeaderTruncated;

bool zip64Needed = false;
Comment thread
carlossanlop marked this conversation as resolved.
Outdated
Expand Down Expand Up @@ -908,7 +908,7 @@ private bool WriteLocalFileHeader(bool isEmptyFile, bool forceWrite)
Debug.Assert(_storedEntryNameBytes.Length <= ushort.MaxValue);

// decide if we need the Zip64 extra field:
Zip64ExtraField zip64ExtraField = default;
Zip64ExtraField zip64ExtraField = new();
bool zip64Used = false;
Comment thread
carlossanlop marked this conversation as resolved.
Outdated
uint compressedSizeTruncated, uncompressedSizeTruncated;

Expand Down
18 changes: 9 additions & 9 deletions src/libraries/System.IO.Compression/src/System/IO/Compression/ZipBlocks.FieldLengths.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

namespace System.IO.Compression
{
internal partial struct ZipGenericExtraField
internal sealed partial class ZipGenericExtraField
{
private static class FieldLengths
{
Expand All @@ -12,7 +12,7 @@ private static class FieldLengths
}
}

internal partial struct Zip64ExtraField
internal sealed partial class Zip64ExtraField
{
internal static class FieldLengths
{
Expand All @@ -23,7 +23,7 @@ internal static class FieldLengths
}
}

internal partial struct Zip64EndOfCentralDirectoryLocator
internal sealed partial class Zip64EndOfCentralDirectoryLocator
{
internal static class FieldLengths
{
Expand All @@ -34,7 +34,7 @@ internal static class FieldLengths
}
}

internal partial struct Zip64EndOfCentralDirectoryRecord
internal sealed partial class Zip64EndOfCentralDirectoryRecord
{
private static class FieldLengths
{
Expand All @@ -51,7 +51,7 @@ private static class FieldLengths
}
}

internal readonly partial struct ZipLocalFileHeader
internal sealed partial class ZipLocalFileHeader
{
internal static class FieldLengths
{
Expand All @@ -67,7 +67,7 @@ internal static class FieldLengths
public const int ExtraFieldLength = sizeof(ushort);
}

internal readonly partial struct ZipDataDescriptor
internal sealed partial class ZipDataDescriptor
{
internal static class FieldLengths
{
Expand All @@ -78,7 +78,7 @@ internal static class FieldLengths
}
}

internal readonly partial struct Zip64DataDescriptor
internal sealed partial class Zip64DataDescriptor
{
internal static class FieldLengths
{
Expand All @@ -90,7 +90,7 @@ internal static class FieldLengths
}
}

internal partial struct ZipCentralDirectoryFileHeader
internal sealed partial class ZipCentralDirectoryFileHeader
{
internal static class FieldLengths
{
Expand All @@ -114,7 +114,7 @@ internal static class FieldLengths
}
}

internal partial struct ZipEndOfCentralDirectoryBlock
internal sealed partial class ZipEndOfCentralDirectoryBlock
{
internal static class FieldLengths
{
Expand Down
18 changes: 9 additions & 9 deletions src/libraries/System.IO.Compression/src/System/IO/Compression/ZipBlocks.FieldLocations.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

namespace System.IO.Compression
{
internal partial struct ZipGenericExtraField
internal sealed partial class ZipGenericExtraField
{
internal static class FieldLocations
{
Expand All @@ -13,7 +13,7 @@ internal static class FieldLocations
}
}

internal partial struct Zip64ExtraField
internal sealed partial class Zip64ExtraField
{
internal static class FieldLocations
{
Expand All @@ -26,7 +26,7 @@ internal static class FieldLocations
}
}

internal partial struct Zip64EndOfCentralDirectoryLocator
internal sealed partial class Zip64EndOfCentralDirectoryLocator
{
private static class FieldLocations
{
Expand All @@ -37,7 +37,7 @@ private static class FieldLocations
}
}

internal partial struct Zip64EndOfCentralDirectoryRecord
internal sealed partial class Zip64EndOfCentralDirectoryRecord
{
private static class FieldLocations
{
Expand All @@ -54,7 +54,7 @@ private static class FieldLocations
}
}

internal readonly partial struct ZipLocalFileHeader
internal sealed partial class ZipLocalFileHeader
{
internal static class FieldLocations
{
Expand All @@ -71,7 +71,7 @@ internal static class FieldLocations
public static readonly int DynamicData = ExtraFieldLength + FieldLengths.ExtraFieldLength;
}

internal readonly partial struct ZipDataDescriptor
internal sealed partial class ZipDataDescriptor
{
internal static class FieldLocations
{
Expand All @@ -82,7 +82,7 @@ internal static class FieldLocations
}
}

internal readonly partial struct Zip64DataDescriptor
internal sealed partial class Zip64DataDescriptor
{
internal static class FieldLocations
{
Expand All @@ -94,7 +94,7 @@ internal static class FieldLocations
}
}

internal partial struct ZipCentralDirectoryFileHeader
internal sealed partial class ZipCentralDirectoryFileHeader
{
internal static class FieldLocations
{
Expand All @@ -119,7 +119,7 @@ internal static class FieldLocations
}
}

internal partial struct ZipEndOfCentralDirectoryBlock
internal sealed partial class ZipEndOfCentralDirectoryBlock
{
private static class FieldLocations
{
Expand Down
Loading