Repository navigation
zlib: use macro list for brotli error constants - #66932
Suryanshpandey07 wants to merge 1 commit into
Conversation
|
Welcome to Node.js, and thank you for your first contribution! Before review, please take a moment to read:
Please make sure every commit is signed off. For a first pull request, GitHub Actions require collaborator approval and Jenkins CI must be started by a collaborator or triager, so an initial wait is normal. |
Expose Brotli decoder error constants using BROTLI_DECODER_ERROR_CODES_LIST instead of manually listing each constant. This avoids compilation errors when building against older shared Brotli libraries (such as Brotli <= 1.2.0) where BROTLI_DECODER_ERROR_FORMAT_BLOCK_SWITCH is not yet defined. Signed-off-by: Suryanshpandey07 <diyanshpandey1234@gmail.com>
25b7c36 to
8fe8164
Compare
ContextWhen building Node.js configured with SolutionBrotli's public header By switching to this macro list:
|
|
There are conflicts in this PR. If you would like to keep this PR active, please rebase the branch in this PR according to the Pull requests documentation. |
Expose Brotli decoder error constants using BROTLI_DECODER_ERROR_CODES_LIST instead of manually listing each constant. This avoids compilation errors when building against older shared Brotli libraries (such as Brotli <= 1.2.0) where BROTLI_DECODER_ERROR_FORMAT_BLOCK_SWITCH is not yet defined.