diff --git a/src/blackmisc/settingscache.cpp b/src/blackmisc/settingscache.cpp
index c22809f16..c32bd7b35 100644
--- a/src/blackmisc/settingscache.cpp
+++ b/src/blackmisc/settingscache.cpp
@@ -37,10 +37,7 @@ namespace BlackMisc
     void CSettingsCache::saveToStoreByPacket(const CValueCachePacket &values)
     {
         CStatusMessage status = saveToFiles(persistentStore(), values.toVariantMap());
-        if (! status.isEmpty())
-        {
-            CLogMessage(this).preformatted(status);
-        }
+        CLogMessage(this).preformatted(status);
     }
 
     BlackMisc::CStatusMessage CSettingsCache::loadFromStore()
diff --git a/src/blackmisc/valuecache.cpp b/src/blackmisc/valuecache.cpp
index 26fd10097..190ca2a30 100644
--- a/src/blackmisc/valuecache.cpp
+++ b/src/blackmisc/valuecache.cpp
@@ -297,7 +297,7 @@ namespace BlackMisc
             }
             CVariantMap temp;
             temp.convertFromJson(json.object());
-            temp.removeByKeyIf([&keys](const QString &key) { return keys.contains(key); }); // TODO optimize by skipping files
+            temp.removeByKeyIf([&keys](const QString & key) { return ! keys.contains(key); }); // TODO optimize by skipping files
             temp.removeDuplicates(currentValues);
             o_values.insert(temp, QFileInfo(file).lastModified().toMSecsSinceEpoch());
         }