Checking with isEmpty() is more effective. Signed-off-by: Markus Heidelberg <markus.heidelberg@xxxxxx> --- src/mainimpl.cpp | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mainimpl.cpp b/src/mainimpl.cpp index 8c7b19e..60414ab 100644 --- a/src/mainimpl.cpp +++ b/src/mainimpl.cpp @@ -1179,7 +1179,7 @@ void MainImpl::doContexPopup(SCRef sha) { act = contextRmtMenu.addAction(*it); act->setData("Ref"); } - if (contextRmtMenu.actions().count() > 0) + if (!contextRmtMenu.isEmpty()) contextMenu.addMenu(&contextRmtMenu); if (!bn.empty()) @@ -1194,7 +1194,7 @@ void MainImpl::doContexPopup(SCRef sha) { act->setData("Ref"); } if (!tn.empty()) - if (contextSubMenu.actions().count() > 0) + if (!contextSubMenu.isEmpty()) contextSubMenu.addSeparator(); else contextMenu.addSeparator(); @@ -1207,7 +1207,7 @@ void MainImpl::doContexPopup(SCRef sha) { act->setData("Ref"); } - if (contextSubMenu.actions().count() > 0) + if (!contextSubMenu.isEmpty()) contextMenu.addMenu(&contextSubMenu); } contextMenu.exec(QCursor::pos()); -- 1.6.3.2.213.g30b07 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html