/algebird

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

removed nop match #22

Merged
merged 1 commit into from Sep 4, 2012

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@mosesn @travisbot @johnynek
@mosesn
Contributor

mosesn commented Sep 3, 2012

motivation

I don't know what I was thinking before, but the right match wasn't doing anything in the first place. It was basically dead code.

changed behavior

None. Just removed some nops.
@travisbot

This comment has been minimized.

Show comment
Hide comment
@travisbot

travisbot Sep 3, 2012

This pull request passes (merged 5771fa5 into 8576918).

travisbot commented Sep 3, 2012

This pull request passes (merged 5771fa5 into 8576918).

johnynek added a commit that referenced this pull request Sep 4, 2012

@johnynek
Merge pull request #22 from mosesn/simplify_folder 
removed nop match
25fead9

@johnynek johnynek merged commit 25fead9 into twitter:develop Sep 4, 2012
1 check passed

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment