Repository navigation
"Split" and "convert" are undefined #71
Description
Activity
in case any text changes in the future, this is referenced in (as of writing):
step 4 in https://compression.spec.whatwg.org/#compress-and-enqueue-a-chunk
step 3 in https://compression.spec.whatwg.org/#compress-flush-and-enqueue
step 4 in https://compression.spec.whatwg.org/#decompress-and-enqueue-a-chunk
step 4 in https://compression.spec.whatwg.org/#decompress-flush-and-enqueueI was thinking of modifying the steps to something like:
- Pass |chunk| to |context|.
- Let |results| be « ».
- While |context| has enough data to decompress:
- Let |buffer| be the result of the next decompression result.
- [=list/Append=] |results| with the result of converting |buffer| to Uint8Array.
It still does not define what "enough data" means, but that can be implementation detail (as it depends on different libraries) and this would match more of what the implementations do.
What does "Pass" mean there?
If something is implementation-defined it would be good to state that explicitly and link the relevant term in Infra.
Reacted by Kagami Sascha RosylightWhat does "Pass" mean there?
Giving the chunk to the underlying compression library to compress/decompress. Maybe there's a better way to phrase that.
If something is implementation-defined it would be good to state that explicitly and link the relevant term in Infra.
Agreed, although not immediately clear to me how it could be defined in terms of infra.
There should probably be some concept of a compression system and then you can define "Pass" for that and link it all up.
Infra defines "implementation-defined" which is what you'd link there.
Reacted by Kagami Sascha RosylightThere should probably be some concept of a compression system and then you can define "Pass" for that and link it all up.
Isn't that what "compression context" supposed to be? Maybe we can add more states there.
What is the issue with the Compression Standard?
In
it's somewhat clear what needs to happen, but generally we define "split" and "convert" in more detail.