Expand allowed data capacity for barcodes if size constraints allow

Open hschimke opened this issue 2 years ago • 2 comments

https://www.reddit.com/r/rust/comments/1117fpk/comment/j8ea409/?utm_source=share&utm_medium=web2x&context=3

Feb 13 '23 20:02 hschimke

Looking at this, it appears that we should support up to version 40. I tested with default settings and adding a 1kb file and RXing-cli errors with “data too large to encode”. I believe that this is due to either the estimate_size or pick_version helpers in the QRCode encoder module.

Feb 16 '23 00:02 hschimke

Needs a test case, it looks as if the provisional test for size might not account for varying ec levels, when no ec level has been specified.

Mar 06 '23 18:03 hschimke