Skip to content

regenerate opdict with lspace=rspace=0 for U+002f issue #260 #265

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

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

davidcarlisle
Copy link
Collaborator

@davidcarlisle davidcarlisle commented Nov 1, 2024

This re-runs the python tables/operator-dictionary.py to regenerate operator-dictionary.html and operator-dictionary-compact.html As expected the only change is that / is moved to category K

As discussed in #260 it does not default / to stretchy (so it matches \ )

This was made to a locally modified unicode.xml changing

         <operator-dictionary priority="680" form="infix" lspace="4" rspace="4"/>

to

         <operator-dictionary priority="680" form="infix" lspace="0" rspace="0"/>

if this PR is merged that change will be pushed to the w3c/xml-entities repository (and reflected in the mathml4 operator dictionary)

@NSoiffer
Copy link
Contributor

NSoiffer commented Nov 2, 2024

What's different in the (large) figure caption diff? Or maybe, "why did that change?" / (0x002f) isn't in that diff.

@davidcarlisle
Copy link
Collaborator Author

What's different in the (large) figure caption diff? Or maybe, "why did that change?" / (0x002f) isn't in that diff.

one entry changed from 0x402F to 0xC02F due to the bit packing of the data into the (changed) categories) @NSoiffer

Copy link
Contributor

@NSoiffer NSoiffer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the clarification. I couldn't see the change.

@davidcarlisle
Copy link
Collaborator Author

Thanks for the clarification. I couldn't see the change.

that's what emacs ediff is for (highlights character level diff over the line based diff...

image

@NSoiffer NSoiffer merged commit 55f9c3b into main Nov 25, 2024
1 check passed
@Krinkle Krinkle deleted the zero-space-slash branch January 27, 2025 20:57
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.

2 participants