Small coding improvements

Signed-off-by: Dries Mys <[email protected]>
nextcloud · Jul 20, 2023 · 55f27cd · 55f27cd
1 parent e9aa976
commit 55f27cd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/gui/folder.cpp b/src/gui/folder.cpp
@@ -564,15 +564,17 @@ void Folder::slotWatchedPathChanged(const QString &path, ChangeReason reason)
     if (pathIsIgnored(path)) {
         const auto pinState = _vfs->pinState(relativePath.toString());
         if (!pinState || *pinState != PinState::Excluded) {
-            if (!_vfs->setPinState(relativePath.toString(), PinState::Excluded))
+            if (!_vfs->setPinState(relativePath.toString(), PinState::Excluded)) {
                 qCWarning(lcFolder) << "Could not set pin state of" << relativePath << "to excluded";
+            }
         }
         return;
     } else {
         const auto pinState = _vfs->pinState(relativePath.toString());
         if (pinState && *pinState == PinState::Excluded) {
-            if (!_vfs->setPinState(relativePath.toString(), PinState::Inherited))
+            if (!_vfs->setPinState(relativePath.toString(), PinState::Inherited)) {
                 qCWarning(lcFolder) << "Could not switch pin state of" << relativePath << "from" << *pinState << "to inherited";
+            }
         }
     }
 

diff --git a/src/libsync/vfs/cfapi/vfs_cfapi.cpp b/src/libsync/vfs/cfapi/vfs_cfapi.cpp
@@ -229,8 +229,9 @@ Result<Vfs::ConvertToPlaceholderResult, QString> VfsCfApi::convertToPlaceholder(
     if (item._type != ItemTypeDirectory && OCC::FileSystem::isLnkFile(filename)) {
         qCInfo(lcCfApi) << "File \"" << filename << "\" is a Windows shortcut. Not converting it to a placeholder.";
         const auto pinState = pinStateLocal(localPath);
-        if (!pinState || *pinState != PinState::Excluded)
+        if (!pinState || *pinState != PinState::Excluded) {
             setPinStateLocal(localPath, PinState::Excluded);
+        }
         return Vfs::ConvertToPlaceholderResult::Ok;
     }