Skip to content

Conversation

@txgruppi
Copy link
Contributor

@txgruppi txgruppi commented Mar 4, 2025

I'm writing some constants for working with a PDF.
A PDF has the operators ' and ", when I tried to create constants for these operators it would generated an invalid file.
The constant value would be correct, like OPMoveToNextLineSetSpacingShowText OP = "\"", which is valid, but in the _OPValue map it would create a key with the value "\\\"", which is invalid.
I updated the code to fix that. It passes all the previous tests and now it generates the proper value and the proper key.

@txgruppi txgruppi requested a review from abice as a code owner March 4, 2025 16:16
Copy link
Owner

@abice abice left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks for the contribution!

@abice abice merged commit a6f63bd into abice:master Mar 10, 2025
2 checks passed
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