8000 MNT: Replace outdated comment with self-explaining code (hatching in pdf backend) by jkseppan · Pull Request #4610 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]
Skip to content

MNT: Replace outdated comment with self-explaining code (hatching in pdf backend) #4610

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 10, 2015
Merged

MNT: Replace outdated comment with self-explaining code (hatching in pdf backend) #4610

merged 1 commit into from
Jul 10, 2015

Conversation

jkseppan
Copy link
Member
@jkseppan jkseppan commented Jul 9, 2015

A refactoring in 2008 made this comment obsolete; instead of updating the comment, make the code simply say what the data is.

tacaswell added a commit that referenced this pull request Jul 10, 2015
@tacaswell
Merge pull request #4610 from jkseppan/pdf-hatch-comment
7535c26 
MNT: Replace outdated comment with self-explaining code (hatching in pdf backend)
@tacaswell tacaswell merged commit 7535c26 into matplotlib:master Jul 10, 2015
@tacaswell
Copy link
Member

Thanks, the new code is definitely more readable!

@jkseppan jkseppan deleted the pdf-hatch-comment branch July 10, 2015 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jkseppan @tacaswell
0