Event loop uses std::priority_queue for timeouts instead of std::multimap

[jbaldwin]

Event loop uses std::priority_queue for timeouts instead of std::multimap

[jbaldwin]

Upon further thought I'm not sure this is possible. The data structure needs stable iterators into the nodes of the map, something priority queue cannot guarantee. It could be possible to keep the nodes that the priority queue point to alive until the timeout triggers and check to see if the response was already received and then throw away the timeout, but this has the disadvantage of the timeout always triggering rather than only triggering if the response hasn't been received yet. It also has the downside of pinning the response object within the event loop until the timeout fires, possibly using more memory and/or lifetime issues with the user of the library. I think its safe to say the overhead of the std::multimap is most likely worth it to avoid all these problems.