Skip to content

Commit

Permalink
Cleanup debug output
Browse files Browse the repository at this point in the history
  • Loading branch information
rhaschke committed Oct 26, 2023
1 parent 059782f commit 65537fc
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 9 deletions.
1 change: 0 additions & 1 deletion core/src/stage.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -865,7 +865,6 @@ template bool ConnectingPrivate::hasPendingOpposites<Interface::BACKWARD>(const
const InterfaceState* start) const;

bool ConnectingPrivate::canCompute() const {
ROS_DEBUG_STREAM("canCompute " << name() << ": " << pendingPairsPrinter());
// Do we still have feasible pending state pairs?
return !pending.empty() && pending.front().first->priority().enabled() &&
pending.front().second->priority().enabled();
Expand Down
13 changes: 5 additions & 8 deletions visualization/motion_planning_tasks/src/remote_task_model.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -501,14 +501,11 @@ QVariant RemoteSolutionModel::data(const QModelIndex& index, int role) const {
return item.id;

case Qt::ToolTipRole:
switch (index.column()) {
#if 1 // show internal solution id in first column return item
case 0:
return item.id;
default:
return item.comment;
#endif
}
// internal solution id in first column
if ((false) && index.column() == 0)

Check failure on line 505 in visualization/motion_planning_tasks/src/remote_task_model.cpp

View workflow job for this annotation

GitHub Actions / noetic-source • clang-tidy

clang-tidy: redundant boolean literal supplied to boolean operator (readability-simplify-boolean-expr)
return item.id;
// usually just show the comment
return item.comment;
break;

case Qt::DisplayRole:
Expand Down

0 comments on commit 65537fc

Please sign in to comment.