Brotli 1.1.0 changed return values? #486

Closed
opened 2026-01-29 20:44:43 +00:00 by claunia · 5 comments
Owner

Originally created by @kmilos on GitHub (Sep 21, 2023).

See https://github.com/Exiv2/exiv2/issues/2762 and https://github.com/Exiv2/exiv2/pull/2763

Originally created by @kmilos on GitHub (Sep 21, 2023). See https://github.com/Exiv2/exiv2/issues/2762 and https://github.com/Exiv2/exiv2/pull/2763
Author
Owner

@eustas commented on GitHub (Sep 21, 2023):

Correct. Previous error descriptions were somewhat ugly.

@eustas commented on GitHub (Sep 21, 2023): Correct. Previous error descriptions were somewhat ugly.
Author
Owner

@eustas commented on GitHub (Sep 21, 2023):

See also CLI changes in 2e6164d7b0

@eustas commented on GitHub (Sep 21, 2023): See also CLI changes in https://github.com/google/brotli/commit/2e6164d7b0b7bdcb5d731bee6dd0f4e06708e006
Author
Owner

@eustas commented on GitHub (Sep 21, 2023):

In the given example "CL_SPACE" does not give any hint on what happened; now it says that this is error in the given input ("format" prefix).

@eustas commented on GitHub (Sep 21, 2023): In the given example "CL_SPACE" does not give any hint on what happened; now it says that this is error in the given input ("format" prefix).
Author
Owner

@kmilos commented on GitHub (Sep 21, 2023):

In the given example "CL_SPACE" does not give any hint on what happened; now it says that this is error in the given input ("format" prefix).

Ta. I guess we'll just have to deal w/ both outputs somehow... 😞

@kmilos commented on GitHub (Sep 21, 2023): > In the given example "CL_SPACE" does not give any hint on what happened; now it says that this is error in the given input ("format" prefix). Ta. I guess we'll just have to deal w/ both outputs somehow... 😞
Author
Owner

@eustas commented on GitHub (Sep 21, 2023):

Simply checking suffix should work...

@eustas commented on GitHub (Sep 21, 2023): Simply checking suffix should work...
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/brotli#486