Don't auto-throw non-viral callee exceptions before callerLambda

This commit is contained in:
2026-06-06 04:47:59 -04:00
parent 8a7d4272bd
commit edde8f4a64
3 changed files with 47 additions and 12 deletions
+3 -6
View File
@@ -11,6 +11,7 @@
#include <spinscale/spinLock.h>
#include <spinscale/co/coQutex.h>
#include <spinscale/co/nonViralCompletion.h>
#include <spinscale/co/promiseChainLink.h>
#include <spinscale/co/promiseReturnOps.h>
#include <spinscale/co/returnValues.h>
@@ -126,12 +127,8 @@ struct NonPostingPromise
<< std::this_thread::get_id()
<< " Non-viral non-posting: invoking callerLambda directly.\n";
#endif
if (calleePromise.returnValues.myExceptionPtr) {
std::rethrow_exception(
calleePromise.returnValues.myExceptionPtr);
}
calleePromise.callerLambda();
auto callerLambda = std::move(calleePromise.callerLambda);
callerLambda();
return std::noop_coroutine();
}
+40
View File
@@ -0,0 +1,40 @@
#ifndef NON_VIRAL_COMPLETION_H
#define NON_VIRAL_COMPLETION_H
#include <exception>
#include <utility>
namespace sscl::co {
class NonViralCompletion
{
public:
explicit NonViralCompletion(std::exception_ptr &exceptionPtr)
: exceptionPtr(exceptionPtr)
{}
bool hasException() const noexcept
{
return exceptionPtr != nullptr;
}
void checkAndRethrowException() const
{
if (exceptionPtr)
{
std::rethrow_exception(exceptionPtr);
}
}
std::exception_ptr releaseException() noexcept
{
return std::exchange(exceptionPtr, nullptr);
}
private:
std::exception_ptr &exceptionPtr;
};
} // namespace sscl::co
#endif // NON_VIRAL_COMPLETION_H
+4 -6
View File
@@ -17,6 +17,7 @@
#include <spinscale/componentThread.h>
#include <spinscale/co/coQutex.h>
#include <spinscale/co/nonViralCompletion.h>
#include <spinscale/co/postTarget.h>
#include <spinscale/co/promiseChainLink.h>
#include <spinscale/co/promiseReturnOps.h>
@@ -164,15 +165,12 @@ struct PostingPromise
std::cout << "final_suspend" << ": " << std::this_thread::get_id()
<< " Non-viral: posting callerLambda completion to callerIoContext.\n";
#endif
auto callerLambda = std::move(calleePromise.callerLambda);
boost::asio::post(
calleePromise.callerIoContext,
[&calleeRef = calleePromise]()
[callerLambda = std::move(callerLambda)]() mutable
{
if (calleeRef.returnValues.myExceptionPtr) {
std::rethrow_exception(calleeRef.returnValues.myExceptionPtr);
}
calleeRef.callerLambda();
callerLambda();
});
}
else