Skip to content

spec: clarify LZMA properties prefix in cmpLength and cmpCrc64 scope#20

Open
kwinsch wants to merge 1 commit intoaaru-dps:develfrom
RomTholos:spec/document-lzma-properties-prefix
Open

spec: clarify LZMA properties prefix in cmpLength and cmpCrc64 scope#20
kwinsch wants to merge 1 commit intoaaru-dps:develfrom
RomTholos:spec/document-lzma-properties-prefix

Conversation

@kwinsch
Copy link
Copy Markdown
Contributor

@kwinsch kwinsch commented Mar 29, 2026

Clarifies two ambiguities for LZMA-compressed blocks (IDs 1, 3):

  • cmpLength includes the 5-byte LZMA properties prefix
  • cmpCrc64 covers only the compressed stream, excluding the prefix

Adds detailed notes to the compression appendix with on-disk layout diagram
and decompression steps. Updates cmpLength/cmpCrc64 field descriptions in
all block specs that carry these fields (data, ddt, ddt2, datastream_payload)
with cross-references to Appendix C.

@kwinsch kwinsch force-pushed the spec/document-lzma-properties-prefix branch from a7401bd to bbac1fd Compare March 29, 2026 17:18
@kwinsch
Copy link
Copy Markdown
Contributor Author

kwinsch commented Mar 29, 2026

Removed merge conflict.

@claunia
Copy link
Copy Markdown
Collaborator

claunia commented Mar 29, 2026

sorry created another conflict

@kwinsch kwinsch force-pushed the spec/document-lzma-properties-prefix branch from bbac1fd to 2cbd88a Compare March 31, 2026 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants