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

[en] Retry decoding tags with errors if " and " in tag #893

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

kristian-clausal
Copy link
Collaborator

@kristian-clausal kristian-clausal commented Oct 29, 2024

Removed unused code with valid_words from 2020. Didn't find an elegant way to use it where it was meant to be used.

I've been hitting my head trying to figure out what to do in decode_tags1 to get " and " to parse correctly (that is, ignored it), and finally gave up. Instead, I've implemented a new kludge in the vein of the "/"- kludge preceding this, except thankfully simpler.

Fixes #891

Removed unused code with `valid_words` from 2020. Didn't
find an elegant way to use it where it was meant to be
used.

I've been hitting my head trying to figure out what
to do in decode_tags1 to get " and " to parse correctly
(that is, ignored it), and finally gave up. Instead,
I've implemented a new kludge in the vein of the "/"-
kludge preceding this, except thankfully simpler.
@kristian-clausal kristian-clausal merged commit 598a4ee into master Oct 29, 2024
10 checks passed
@kristian-clausal kristian-clausal deleted the formof branch October 29, 2024 09:46
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.

[en] Decode tags algorithm broken with "x and y" style constructions
1 participant