Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent --raw CLR option from failing if result.getRawBytes() is null #1683

Merged
merged 1 commit into from Sep 7, 2023
Merged

Prevent --raw CLR option from failing if result.getRawBytes() is null #1683

merged 1 commit into from Sep 7, 2023

Conversation

dlenski
Copy link
Contributor

@dlenski dlenski commented Sep 7, 2023

Fixes #1682

@srowen
Copy link
Contributor

srowen commented Sep 7, 2023

Looks good, thanks!

@srowen srowen merged commit 672bd40 into zxing:master Sep 7, 2023
5 checks passed
@srowen srowen added this to the 3.5.3 milestone Sep 7, 2023
@srowen srowen added the bug label Sep 7, 2023
@dlenski dlenski deleted the patch-1 branch September 7, 2023 20:31
dlenski added a commit to dlenski/python-zxing that referenced this pull request Nov 2, 2023
(This probably should have been done as part of 0512c7a.)

The --raw option was added in 3.5.0, but broken for certain barcode types
(PDF_417 and maybe others) until 3.5.3: CommandLineRunner will fail with
NullPointerException.

See zxing/zxing#1682 and
zxing/zxing#1683 for more details.
dlenski added a commit to dlenski/python-zxing that referenced this pull request Feb 20, 2024
Among other improvements, this release of ZXing fixes the brokenness of the
`--raw` option (added in 3.5.0, but broken for `PDF_417` and maybe other
barcode types). See my bug report and fix for that issue at
zxing/zxing#1682 and
zxing/zxing#1683.
dlenski added a commit to dlenski/python-zxing that referenced this pull request Feb 20, 2024
Among other improvements, this release of ZXing fixes the brokenness of the
`--raw` option (added in 3.5.0, but broken for `PDF_417` and maybe other
barcode types). See my bug report and fix for that issue at
zxing/zxing#1682 and
zxing/zxing#1683.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

CommandLineRunner --raw option is broken for PDF_417 barcodes
2 participants