diff options
| author | Stefan Radomski <github@mintwerk.de> | 2017-05-18 13:12:36 (GMT) |
|---|---|---|
| committer | Stefan Radomski <github@mintwerk.de> | 2017-05-18 13:12:36 (GMT) |
| commit | 7df563d41c9e76c0946aa6de0f542e33d9586946 (patch) | |
| tree | 5398c1659bc9b2c9b88d107b63b20a18c0eb2f91 /src | |
| parent | 3305444946481c09a239735b3b8f6ca92f1ed38f (diff) | |
| download | uscxml-7df563d41c9e76c0946aa6de0f542e33d9586946.zip uscxml-7df563d41c9e76c0946aa6de0f542e33d9586946.tar.gz uscxml-7df563d41c9e76c0946aa6de0f542e33d9586946.tar.bz2 | |
Fixed issue 132
Diffstat (limited to 'src')
| -rw-r--r-- | src/uscxml/interpreter/BasicEventQueue.cpp | 17 |
1 files changed, 16 insertions, 1 deletions
diff --git a/src/uscxml/interpreter/BasicEventQueue.cpp b/src/uscxml/interpreter/BasicEventQueue.cpp index 5b3e9f8..32c6d0a 100644 --- a/src/uscxml/interpreter/BasicEventQueue.cpp +++ b/src/uscxml/interpreter/BasicEventQueue.cpp @@ -34,7 +34,13 @@ BasicEventQueue::~BasicEventQueue() { Event BasicEventQueue::dequeue(size_t blockMs) { std::lock_guard<std::recursive_mutex> lock(_mutex); - if (blockMs > 0) { + if (blockMs == std::numeric_limits<size_t>::max()) { + // handle "forever" explicitly, see comments below + while (_queue.empty()) { + _cond.wait(_mutex); + } + + } else if (blockMs > 0) { using namespace std::chrono; // TODO: do read http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2008/n2661.htm @@ -43,6 +49,15 @@ Event BasicEventQueue::dequeue(size_t blockMs) { // now + milliseconds(blockMs) may not have fitted into a duration type - limit to maximum duration if (blockMs > (size_t)(system_clock::duration::max().count() - duration_cast<milliseconds>(now.time_since_epoch()).count())) { + /** + * Some compilers do not like to wait_until system_clock::time_point::max(): + * https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58931 + * http://stackoverflow.com/questions/39041450/stdcondition-variable-wait-until-surprising-behaviour + * + * See also: + * https://github.com/tklab-tud/uscxml/issues/101 + * https://github.com/tklab-tud/uscxml/issues/132 + */ endTime = system_clock::time_point::max(); } |