Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Minor polish for fallThru handling #447

Closed
wants to merge 1 commit into from
Closed

Minor polish for fallThru handling #447

wants to merge 1 commit into from

Conversation

hzlmn
Copy link
Contributor

@hzlmn hzlmn commented Mar 1, 2017 •
edited
Loading

There are no reasons to re-assing value to already created fallThru

@boopathi
Copy link
Member

boopathi commented Mar 1, 2017

This code will be changed to fix for #423. This will not be required.

@hzlmn
Copy link
Contributor Author

hzlmn commented Mar 1, 2017

Are you already in-progress with that one?

@hzlmn
Copy link
Contributor Author

hzlmn commented Mar 2, 2017

I am asking because I can look on #423 and #448 if not.

@boopathi
Copy link
Member

boopathi commented Mar 2, 2017

Sure. Feel free to debug them. I'm kinda busy with other things.

@hzlmn
Copy link
Contributor Author

hzlmn commented Mar 3, 2017

closing it, will be improved in next PR.

@hzlmn hzlmn closed this Mar 3, 2017
# for free to join this conversation on GitHub. Already have an account? # 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
@hzlmn @boopathi