roc-streaming / roc-toolkit

Real-time audio streaming over the network.
https://roc-streaming.org
Mozilla Public License 2.0
1.09k stars 213 forks source link

Updated insert_sleeping_task function to try to improve the complexity time #781

Closed JimmyL04 closed 1 month ago

JimmyL04 commented 1 month ago

This PR tried improves the time complexity of the ControlTaskQueue::insert_sleepingtask function by replacing the sorted linked list used for delayed tasks with a std::priority_queue.

Key Improvements:

Changes Made:

github-actions[bot] commented 1 month ago

:robot: Upon creation, pull request description does not have a link to an issue. If there is a related issue, please add it to the description using any of the supported formats.

gavv commented 1 month ago

Hi,

Besides that it doesn't compile and doesn't have actual code (?), please check out discussion and requirements in the original issue for why std::priority_queue is not an option here: #379