diff --git a/src/Gui/TaskView/TaskWatcher.cpp b/src/Gui/TaskView/TaskWatcher.cpp
index aeb215dbf4..d521ec892f 100644
--- a/src/Gui/TaskView/TaskWatcher.cpp
+++ b/src/Gui/TaskView/TaskWatcher.cpp
@@ -92,7 +92,6 @@ TaskWatcherCommands::TaskWatcherCommands(const char* Filter,const char* commands
             (BitmapFactory().pixmap(pixmap), trUtf8(name), true, 0);
 
         for (const char** i=commands;*i;i++) {
-            if(!i) break;
             Command *c = mgr.getCommandByName(*i);
             if (c) {
                 // handled in TaskBox::actionEvent()
diff --git a/src/Gui/ViewProvider.cpp b/src/Gui/ViewProvider.cpp
index 7aeaacabd8..0c4147ce5d 100644
--- a/src/Gui/ViewProvider.cpp
+++ b/src/Gui/ViewProvider.cpp
@@ -539,7 +539,7 @@ bool ViewProvider::onDelete(const vector< string >& subNames) {
     bool del = true;
     auto vector = getExtensionsDerivedFromType<Gui::ViewProviderExtension>();
     for(Gui::ViewProviderExtension* ext : vector)
-        del = del || ext->extensionOnDelete(subNames);
+        del &= ext->extensionOnDelete(subNames);
 
     return del;
 }