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
Added priority-queue lifo scheduler #2483
Conversation
hkaiser
commented
Feb 5, 2017
- flyby: removed unneeded #includes
- flyby: removed unneeded type definitions
- flyby: removed unneeded #includes - flyby: removed unneeded type definitions
be8c54b
to
57f3d8d
Compare
|
||
// define the default scheduler to use | ||
typedef fifo_priority_queue_scheduler queue_scheduler; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are those not needed anymore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those have never been used. I removed them to avoid instantiating the scheduler types in each translation unit which pulls this #include file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't all the other schedulers be removed from there as well then?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The others are not typedef'ed, but forward declared only.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I was just thinking since they aren't used from those forward declarations anyway, just to avoid unneeded code in the headers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, fair point. I'll try to do that, let's see how it goes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks