Skip to content
This repository was archived by the owner on Feb 23, 2018. It is now read-only.

Conversation

soc
Copy link

@soc soc commented May 20, 2011

This should fix #4467 and a huge amount of the formatting problems around #4490.

@acruise
Copy link

acruise commented Jun 6, 2011

If this could be split up into multiple smaller patches, it would be less controversial. For example, one patch that only changes foo' (incorrect syntax, breaks formatting pretty badly) to foo` (correct syntax but may not precisely match the original intent of the long-ago author).

Ironically, I had to edit this post to escape the backticks. :)

@acruise
Copy link

acruise commented Jun 6, 2011

I would suggest avoiding the capitalization and line-break changes for now, they make the patch much bigger and don't really add much value.

@soc soc closed this Jun 19, 2011
@soc
Copy link
Author

soc commented Jun 19, 2011

This has been pulled in trunk. Thanks to kzys and dubochet!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants