Skip to content

Commit b228046

Browse files
facontidavideclaude
andcommitted
Fix clang-tidy readability-implicit-bool-conversion warnings
Use explicit != nullptr comparisons for pointer-to-bool conversions flagged by clang-tidy. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent b3b0217 commit b228046

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/bt_factory.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -486,7 +486,7 @@ void Tree::remapManifestPointers()
486486
for(auto& node : subtree->nodes)
487487
{
488488
const auto* old_manifest = node->config().manifest;
489-
if(old_manifest)
489+
if(old_manifest != nullptr)
490490
{
491491
auto it = manifests.find(old_manifest->registration_ID);
492492
if(it != manifests.end())

src/tree_node.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -481,7 +481,7 @@ AnyPtrLocked BT::TreeNode::getLockedPortContent(const std::string& key)
481481
{
482482
const auto bb_key = std::string(*remapped_key);
483483
auto result = _p->config.blackboard->getAnyLocked(bb_key);
484-
if(!result && _p->config.manifest)
484+
if(!result && _p->config.manifest != nullptr)
485485
{
486486
// Entry doesn't exist yet. Create it using the port's type info
487487
// from the manifest so that getLockedPortContent works even when

0 commit comments

Comments
 (0)