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.

Sign up for GitHub

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

Jump to bottom

Proposed fixes to #40 #41

Merged

dlenski merged 1 commit into dlenski:master from SeanIsYoung:master

Jul 26, 2024

SeanIsYoung commented Jul 15, 2024

Fixes for the errors in #40.

Error 1: suggested solution is to use l.strip().endswith(b': No barcode found'). The call to .strip() removes the end of line formatting all together and so it should work just fine on either OS.
Error2: suggested solution is to replace b'file:///' with b'file://' as technically b'file:///' is just a shorthand for b'file://localhost/'


          Proposed fixes to [dlenski#40](dlenski#40).

88e302a

SeanIsYoung changed the title ~~Proposed fixes to https://github.com/dlenski/python-zxing/issues/40~~ Proposed fixes to #40

dlenski merged commit ae1d4b6 into dlenski:master

13 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet