Skip to content
Snippets Groups Projects
Verified Commit 494b556b authored by Timm Fitschen's avatar Timm Fitschen
Browse files

Merge branch 'f-files' into f-consolidation

parents ac18a98e 8eaa4281
No related branches found
No related tags found
1 merge request!12F consolidation
...@@ -270,7 +270,8 @@ auto Transaction::Execute() -> TransactionStatus { ...@@ -270,7 +270,8 @@ auto Transaction::Execute() -> TransactionStatus {
return status; return status;
} }
auto Transaction::ExecuteAsynchronously() noexcept -> StatusCode { // TODO(tf) This has apparently a cognitive complexity of 39>25 (threshold).
auto Transaction::ExecuteAsynchronously() noexcept -> StatusCode { // NOLINT
if (!IsStatus(TransactionStatus::READY()) && if (!IsStatus(TransactionStatus::READY()) &&
!IsStatus(TransactionStatus::GO_ON())) { !IsStatus(TransactionStatus::GO_ON())) {
return StatusCode::TRANSACTION_STATUS_ERROR; return StatusCode::TRANSACTION_STATUS_ERROR;
...@@ -371,7 +372,8 @@ auto Transaction::ExecuteAsynchronously() noexcept -> StatusCode { ...@@ -371,7 +372,8 @@ auto Transaction::ExecuteAsynchronously() noexcept -> StatusCode {
return StatusCode::EXECUTING; return StatusCode::EXECUTING;
} }
auto Transaction::WaitForIt() const noexcept -> TransactionStatus { // TODO(tf) This has apparently a cognitive complexity of 36>25 (threshold).
auto Transaction::WaitForIt() const noexcept -> TransactionStatus { // NOLINT
if (this->status.GetCode() != StatusCode::EXECUTING) { if (this->status.GetCode() != StatusCode::EXECUTING) {
return this->status; return this->status;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment