astitcher opened a new pull request, #422: URL: https://github.com/apache/qpid-proton/pull/422
Previously we checked whether the tasks were active without locking which was bad. This is is an alternative fix (which has the correct semantics) to #421. I think it probably still needs a test for the fixed behaviour and the correct semantics before merging. This would be new tests in ```container_test.cpp``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org
