tukaani-project / xz

XZ Utils
https://tukaani.org/xz/
Other
532 stars 95 forks source link

liblzma: Clarify lzma_lzma_preset() documentation in lzma12.h. #40

Closed JiaT75 closed 1 year ago

JiaT75 commented 1 year ago

lzma_lzma_preset() does not guarentee that the lzma_options_lzma are usable in an encoder even if it returns false (success). If liblzma is built with default configurations, then the options will always be usable. However if the match finders hc3, hc4, or bt4 are disabled, then the options may not be usable depending on the preset level requested.

The documentation was updated to reflect this complexity, since this behavior was unclear before.

Pull request checklist

Please check if your PR fulfills the following requirements:

Pull request type

Please check the type of change your PR introduces: - [ ] Bugfix - [ ] Feature - [ ] Code style update (formatting, renaming, typo fix) - [ ] Refactoring (no functional changes, no api changes) - [ ] Build related changes - [X] Documentation content changes - [ ] Other (please describe):

Related Issue URL: fixes https://github.com/tukaani-project/xz/issues/37

Does this introduce a breaking change?