Skip to content

Raise on missing closing quotes in matplotlibrc#22668

Merged
oscargus merged 1 commit intomatplotlib:mainfrom
timhoffm:fix-rcparam-quotes
Mar 21, 2022
Merged

Raise on missing closing quotes in matplotlibrc#22668
oscargus merged 1 commit intomatplotlib:mainfrom
timhoffm:fix-rcparam-quotes

Conversation

@timhoffm
Copy link
Copy Markdown
Member

PR Summary

Closes #22636.

@timhoffm timhoffm force-pushed the fix-rcparam-quotes branch from aa365ca to 89eb2d1 Compare March 18, 2022 22:55
@timhoffm timhoffm added this to the v3.6.0 milestone Mar 18, 2022
@tacaswell
Copy link
Copy Markdown
Member

Are we sure we have no legitimate value which will have one " in it ?

@timhoffm
Copy link
Copy Markdown
Member Author

Single " should be escaped: "Use backslash to escape \" in a quoted string".

This is e.g. also the behavior in YAML, which supports quoted and unquoted strings.

@oscargus
Copy link
Copy Markdown
Member

Should the error message say that? (Espace single ").

@timhoffm timhoffm force-pushed the fix-rcparam-quotes branch from 89eb2d1 to c44ebd7 Compare March 21, 2022 10:48
@timhoffm
Copy link
Copy Markdown
Member Author

Added a note on escaping to the exception message.

@oscargus oscargus merged commit 011ac81 into matplotlib:main Mar 21, 2022
@timhoffm timhoffm deleted the fix-rcparam-quotes branch March 21, 2022 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Infinite loop when there is single double quote in matplotlibrc

3 participants