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

New test cases #272

Closed
wants to merge 1 commit into from
Closed

Conversation

lyngklip
Copy link
Contributor

@lyngklip lyngklip commented Oct 5, 2024

A few test cases that I came up with while reviewing code128. Reading postscript is like listening to Yoda. Thank you for this interesting project.

@terryburton terryburton mentioned this pull request Oct 9, 2024
gitlost added a commit to gitlost/postscriptbarcode that referenced this pull request Oct 12, 2024
Latch/unlatch extended on 4 chars instead of 5 for better
encodation in certain cases (and no pessimizations found so far),
part addresses PR bwipp#272, props lyngklip

Also fix masked extended chars matching ASCII digits in `numsscr`
(skipped using re-synced `numEA` array)
gitlost added a commit to gitlost/postscriptbarcode that referenced this pull request Oct 12, 2024
Adapted from ZXing (props Alex Geller) - maybe 80% slower depending
on data & stack heavy but does improve some outcomes when FNC1s
present (GS1 or manual) although not much else it appears (the
previous algorithm was pretty good)

Prompted by tests added from PR bwipp#272, props lyngklip
@lyngklip lyngklip closed this Oct 26, 2024
@lyngklip lyngklip deleted the tests-for-pull-request branch November 3, 2024 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant