]> cloud.milkyroute.net Git - dolphin.git/commitdiff
dolphintabwidget.cpp piority of boolean operators
authorMarius P <nmariusp1@gmail.com>
Sat, 9 Jul 2022 12:40:28 +0000 (15:40 +0300)
committerMarius P <nmariusp1@gmail.com>
Sat, 9 Jul 2022 12:40:28 +0000 (15:40 +0300)
Fix PVS-Studio issue "warning: V648 Priority of the '&&' operation is higher than that of the '||' operation.".
Fix clangd issue.

src/dolphintabwidget.cpp

index de7a18db3622636385910225d8af5b51ae090b6c..8eac9519d289830e2a710d6d4a38a2ccc9610bad 100644 (file)
@@ -490,13 +490,13 @@ QPair<int, bool> DolphinTabWidget::indexByUrl(const QUrl& url, ChildUrlBehavior
     do {
         const auto tabPage = tabPageAt(i);
         if (tabPage->primaryViewContainer()->url() == url ||
-                childUrlBehavior == ReturnIndexForOpenedParentAlso && tabPage->primaryViewContainer()->url().isParentOf(url)) {
+                (childUrlBehavior == ReturnIndexForOpenedParentAlso && tabPage->primaryViewContainer()->url().isParentOf(url))) {
             return qMakePair(i, true);
         }
 
         if (tabPage->splitViewEnabled() &&
                 (url == tabPage->secondaryViewContainer()->url() ||
-                 childUrlBehavior == ReturnIndexForOpenedParentAlso && tabPage->secondaryViewContainer()->url().isParentOf(url))) {
+                 (childUrlBehavior == ReturnIndexForOpenedParentAlso && tabPage->secondaryViewContainer()->url().isParentOf(url)))) {
             return qMakePair(i, false);
         }