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

mathtext: Fix comma behaviour at start of string #5882

Merged
merged 2 commits into from Jan 25, 2016

Conversation

zblz
Copy link
Member

@zblz zblz commented Jan 19, 2016

The implementation of the comma between braces in #5813 introduced a bug when a comma is at the beginning of the string: e.g., $,10$ (prev_char would not be defined, and everything crashed). This PR fixes this bug, and modifies test 80 so that it covers the scenario of comma at start of string.

@zblz
Copy link
Member Author

zblz commented Jan 25, 2016

ping @mdboom

@mdboom mdboom added this to the Critical bug fix release (1.5.2) milestone Jan 25, 2016
mdboom added a commit that referenced this pull request Jan 25, 2016
mathtext: Fix comma behaviour at start of string
@mdboom mdboom merged commit cc1b0c9 into matplotlib:master Jan 25, 2016
@mdboom mdboom modified the milestones: next bug fix release (2.0.1), Critical bug fix release (1.5.2) Jan 25, 2016
@zblz zblz deleted the edge-comma-dots branch January 25, 2016 17:30
QuLogic pushed a commit to QuLogic/matplotlib that referenced this pull request Oct 16, 2016
mathtext: Fix comma behaviour at start of string
@QuLogic
Copy link
Member

QuLogic commented Oct 16, 2016

Backported to v2.x via b04c5b9.

@QuLogic QuLogic modified the milestones: 2.0.1 (next bug fix release), 2.0 (style change major release) Dec 7, 2016
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.

None yet

3 participants