Skip to content
This repository has been archived by the owner on Apr 8, 2020. It is now read-only.

Remove some unused variables #82

Merged
merged 1 commit into from
Mar 8, 2017
Merged

Remove some unused variables #82

merged 1 commit into from
Mar 8, 2017

Conversation

dshil
Copy link
Contributor

@dshil dshil commented Mar 7, 2017

No description provided.

@dshil dshil requested a review from eugulixes March 7, 2017 03:52
@eugulixes
Copy link
Contributor

eugulixes commented Mar 8, 2017 •
edited
Loading

@dshil, the commit is very useful, but I think it should be renamed into something like Remove some unused variables because code is very common term. When I saw the commit message for the very first time I imagined removing lines of code.

@dshil
Copy link
Contributor Author

dshil commented Mar 8, 2017

@eugulixes, good point, I've update the PR.

@eugulixes eugulixes changed the title Remove unused code Remove some unused variables Mar 8, 2017
@eugulixes eugulixes merged commit b73c410 into master Mar 8, 2017
@eugulixes eugulixes deleted the remove-unused-code branch March 8, 2017 12:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@eugulixes eugulixes eugulixes approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dshil @eugulixes