Spelt / ZXing.Delphi

ZXing Barcode Scanning object Pascal Library for Delphi VCL and Delphi Firemonkey
Apache License 2.0
479 stars 204 forks source link

QR-code of one-digit numeric string is not recognized #11

Closed ashumkin closed 6 years ago

ashumkin commented 8 years ago

This issue is observed when I was fixing #9. If a string is one-digit only. It is not recognized. "Version number needs to be between 1 and 40" exception is occured.

I don't know (for now) how to correclty fix it.

Spelt commented 8 years ago

Ok Alexey thanks!

I will also take a look at he pull request later this week. This issue is a minor one.

Op 1 mrt. 2016 om 13:54 heeft Alexey Shumkin notifications@github.com het volgende geschreven:

This issue is observed when I was fixing #9. If a string is one-digit only. It is not recognized. "Version number needs to be between 1 and 40" exception is occured.

I don't know (for now) how to correclty fix it.

— Reply to this email directly or view it on GitHub.

Spelt commented 6 years ago

Hi Alexey

I uploaded v3.5 and it looks like this issue is fixed too.

Can you verify this and report back?

ashumkin commented 6 years ago

Hi, Edward! I've verified. It seems commit 168db6f769ef67841e5fadcbf995fe4e208caa2e fixes this issue. Thank you. I'm gonna PR a test for this.

ashumkin commented 6 years ago

I close the issue. It is fixed with the 168db6f769ef67841e5fadcbf995fe4e208caa2e