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

Cleaned up middle a bit. #7315

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@sstewartgallus
Copy link
Contributor

sstewartgallus commented Jun 23, 2013

The only really tricky change is that a long chain of ifs, and elses
was turned into a single if, and a match in astencode.rs. Some methods
can only be called in certain cases, and so have to come after the if.

Cleaned up middle a bit.
The only really tricky change is that a long chain of ifs, and elses
was turned into a single if, and a match in astencode.rs. Some methods
can only be called in certain cases, and so have to come after the if.
@cmr

This comment has been minimized.

Copy link

cmr commented on 0f55c9c Jun 23, 2013

r+ very nice!

@brson

This comment has been minimized.

Copy link
Contributor

brson commented Jun 24, 2013

Taking this on my incoming branch.

@brson brson closed this Jun 24, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment