Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[chore] Cleanup settings code style as reported by #6588. #6709

Merged
merged 1 commit into from
Mar 31, 2016
Merged

[chore] Cleanup settings code style as reported by #6588. #6709

merged 1 commit into from
Mar 31, 2016

Conversation

bevacqua
Copy link
Contributor

  • Removes an inconsecuential assignment
  • Replaces void 0 for a more clear undefined reference
  • Removes unnecessary parenthesis

@bevacqua bevacqua force-pushed the chore/settings-codestyle branch from 084b95d to 7b43d52 Compare March 30, 2016 20:11
@epixa
Copy link
Contributor

epixa commented Mar 31, 2016

LGTM

@epixa epixa assigned bevacqua and unassigned epixa Mar 31, 2016
@bevacqua bevacqua merged commit 26cedc7 into elastic:master Mar 31, 2016
@bevacqua bevacqua deleted the chore/settings-codestyle branch March 31, 2016 17:40
@bevacqua bevacqua removed their assignment Mar 31, 2016
@epixa
Copy link
Contributor

epixa commented Mar 31, 2016

@bevacqua Can you try to backport this to 4.x and 4.5? If it backports cleanly, great. If it doesn't, don't worry about it.

elastic-jasper added a commit that referenced this pull request Mar 31, 2016
@elastic-jasper
Backport PR #6709
f456be9 
---------

**Commit 1:**
[chore] Cleanup settings code style as reported. Fixes #6588.

* Original sha: 7b43d52
* Authored by Nicolas Bevacqua <[email protected]> on 2016-03-30T20:11:35Z
elastic-jasper added a commit that referenced this pull request Mar 31, 2016
@elastic-jasper
Backport PR #6709
a13ab16 
---------

**Commit 1:**
[chore] Cleanup settings code style as reported. Fixes #6588.

* Original sha: 7b43d52
* Authored by Nicolas Bevacqua <[email protected]> on 2016-03-30T20:11:35Z
This was referenced Mar 31, 2016
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
review v5.0.0-alpha1 v5.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bevacqua @epixa