Skip to content

Commit

Permalink
Merge pull request #425 from LeStarch/issue-424
Browse files Browse the repository at this point in the history
Fixing removing the nullptr from join calls
  • Loading branch information
bocchino authored May 23, 2024
2 parents 5de9bc1 + afa074e commit 697603c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -255,7 +255,7 @@ case class TopHelperFns(
getCodeLinesForPhase (CppWriter.Phases.freeThreads) (ci).getOrElse {
if (isActive(ci)) {
val name = getNameAsIdent(ci.qualifiedName)
lines(s"(void) $name.ActiveComponentBase::join(nullptr);")
lines(s"(void) $name.ActiveComponentBase::join();")
}
else Nil
}
Expand Down
4 changes: 2 additions & 2 deletions compiler/tools/fpp-to-cpp/test/top/BasicTopologyAc.ref.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -110,9 +110,9 @@ namespace M {
}

void freeThreads(const TopologyState& state) {
(void) active1.ActiveComponentBase::join(nullptr);
(void) active1.ActiveComponentBase::join();
active2.freeSpecial();
(void) active3.ActiveComponentBase::join(nullptr);
(void) active3.ActiveComponentBase::join();
}

void tearDownComponents(const TopologyState& state) {
Expand Down

0 comments on commit 697603c

Please sign in to comment.