Skip to content

feat: Rewrite zip64 extra - #913

Merged
Its-Just-Nans merged 4 commits into
masterfrom
rewrite-zip64-extra
Aug 11, 2026
Merged

Its-Just-Nans merged 4 commits into
masterfrom
rewrite-zip64-extra

Conversation

@Its-Just-Nans

@Its-Just-Nans Its-Just-Nans commented Aug 10, 2026 •

Copy link
Copy Markdown
Member

pub(crate) sizes: Option<Zip64Sizes>,
pub(crate) header_start: Option<u64>,
// Not used field
// TODO: (unsupported for now)

@amazon-q-developer amazon-q-developer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The refactoring to introduce the Zip64Sizes struct is well-implemented and consistent across all affected files. The changes maintain the same functionality while improving code organization by grouping related fields together. All pattern matching and write/read operations have been properly updated to work with the new structure.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@Its-Just-Nans
Its-Just-Nans added this pull request to the merge queue Aug 11, 2026
@Its-Just-Nans Its-Just-Nans mentioned this pull request Aug 11, 2026
2 tasks done
Merged via the queue into master with commit 63d4180 Aug 11, 2026
134 checks passed
@Its-Just-Nans
Its-Just-Nans deleted the rewrite-zip64-extra branch August 11, 2026 16:53
This was referenced Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants