Skip to content

Conversation

@gdiggs
Copy link
Contributor

@gdiggs gdiggs commented Mar 7, 2016

This seems to be a micro-optimization and merge! is generally cleaner here.

This seems to be a micro-optimization and `merge!` is generally cleaner here.
@wfleming
Copy link
Contributor

wfleming commented Mar 7, 2016

👍

gdiggs added a commit that referenced this pull request Mar 7, 2016
@gdiggs gdiggs merged commit c944b16 into master Mar 7, 2016
@gdiggs gdiggs deleted the GordonDiggs-patch-1 branch March 7, 2016 22:59
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.

2 participants