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

msg: removed unneeded includes from Dispatcher #6814

Merged
merged 1 commit into from Dec 19, 2015

Conversation

stiopaa1
Copy link
Contributor

@stiopaa1 stiopaa1 commented Dec 6, 2015

Removed includes and added forward declarations instead.

Signed-off-by: Michal Jarzabek stiopa@gmail.com

@stiopaa1 stiopaa1 force-pushed the rem_inc branch 2 times, most recently from c5ca5e6 to b387305 Compare December 6, 2015 21:54
@ghost
Copy link

ghost commented Dec 7, 2015

@stiopaa1 please ignore the bot failure, it is a false negative (see http://tracker.ceph.com/issues/13997 for more information). You can re-schedule a job by rebasing your branch and repushing.

Removed includes and added forward declarations instead.

Signed-off-by: Michal Jarzabek <stiopa@gmail.com>
liewegas added a commit that referenced this pull request Dec 19, 2015
msg: removed unneeded includes from Dispatcher

Reviewed-by: Sage Weil <sage@redhat.com>
@liewegas liewegas merged commit 5f9f986 into ceph:master Dec 19, 2015
@ghost ghost changed the title msg/Dispatcher.h:removed unneeded includes msg: removed unneeded includes from Dispatcher Feb 10, 2016
@ghost ghost added the core label Feb 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants