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.

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

xio: remove unused variable #8023

Merged
merged 1 commit into from Mar 11, 2016
Merged

xio: remove unused variable #8023

merged 1 commit into from Mar 11, 2016

Conversation

roidayan
Copy link
Contributor

Signed-off-by: Roi Dayan roid@mellanox.com

Signed-off-by: Roi Dayan <roid@mellanox.com>
@tchaikov
Copy link
Contributor

lgtm

tchaikov added a commit that referenced this pull request Mar 11, 2016
xio: remove unused variable

Reviewed-by: Matt Benjamin <mbenjamin@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov merged commit 9e86b53 into ceph:master Mar 11, 2016
@roidayan roidayan deleted the xio-remove_unused_variable branch March 11, 2016 15:02
@mattbenjamin
Copy link
Contributor

@roidayan should this not be merged?

@roidayan
Copy link
Contributor Author

hi @mattbenjamin , it should and I saw it got merged. I deleted the branch after the merge as it's not needed anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants