Fixing misleading error and equality typo for dependency

Author: Neko-Box-Coder

Src/runcpp2/Data/DependencySource.cpp

@@ -104,10 +104,7 @@ bool runcpp2::Data::DependencySource::Equals(const DependencySource& other) cons
             return git->Equals(*otherGit);
         }
         else
-        {
-            ssLOG_ERROR("Invalid DependencySource type");
             return false;
-        }
     }
     else if(mpark::get_if<LocalSource>(&Source))
     {
@@ -118,10 +115,7 @@ bool runcpp2::Data::DependencySource::Equals(const DependencySource& other) cons
             return local->Equals(*otherLocal);
         }
         else
-        {
-            ssLOG_ERROR("Invalid DependencySource type");
             return false;
-        }
     }
 
     ssLOG_ERROR("Invalid DependencySource type");

Src/runcpp2/Data/ScriptInfo.cpp

@@ -383,15 +383,15 @@ bool runcpp2::Data::ScriptInfo::Equals(const ScriptInfo& other) const
         Cleanup.size() != other.Cleanup.size() ||
         Populated != other.Populated)
     {
-        return true;
+        return false;
     }
 
     for(const auto& it : RequiredProfiles)
     {
         if( other.RequiredProfiles.count(it.first) == 0 || 
             other.RequiredProfiles.at(it.first) != it.second)
         {
-            return true;
+            return false;
         }
     }
 
@@ -400,7 +400,7 @@ bool runcpp2::Data::ScriptInfo::Equals(const ScriptInfo& other) const
         if( other.OverrideCompileFlags.count(it.first) == 0 || 
             !other.OverrideCompileFlags.at(it.first).Equals(it.second))
         {
-            return true;
+            return false;
         }
     }
 
@@ -409,7 +409,7 @@ bool runcpp2::Data::ScriptInfo::Equals(const ScriptInfo& other) const
         if( other.OverrideLinkFlags.count(it.first) == 0 || 
             !other.OverrideLinkFlags.at(it.first).Equals(it.second))
         {
-            return true;
+            return false;
         }
     }
 
@@ -418,7 +418,7 @@ bool runcpp2::Data::ScriptInfo::Equals(const ScriptInfo& other) const
         if( other.OtherFilesToBeCompiled.count(it.first) == 0 || 
             !other.OtherFilesToBeCompiled.at(it.first).Equals(it.second))
         {
-            return true;
+            return false;
         }
     }
 
@@ -427,44 +427,44 @@ bool runcpp2::Data::ScriptInfo::Equals(const ScriptInfo& other) const
         if( other.IncludePaths.count(it.first) == 0 || 
             !other.IncludePaths.at(it.first).Equals(it.second))
         {
-            return true;
+            return false;
         }
     }
 
     for(size_t i = 0; i < Dependencies.size(); ++i)
     {
         if(!Dependencies[i].Equals(other.Dependencies[i]))
-            return true;
+            return false;
     }
 
     for(const auto& it : Defines)
     {
         if(other.Defines.count(it.first) == 0 || !other.Defines.at(it.first).Equals(it.second))
-            return true;
+            return false;
     }
 
     for(const auto& it : Setup)
     {
         if(other.Setup.count(it.first) == 0 || !other.Setup.at(it.first).Equals(it.second))
-            return true;
+            return false;
     }
 
     for(const auto& it : PreBuild)
     {
         if(other.PreBuild.count(it.first) == 0 || !other.PreBuild.at(it.first).Equals(it.second))
-            return true;
+            return false;
     }
 
     for(const auto& it : PostBuild)
     {
         if(other.PostBuild.count(it.first) == 0 || !other.PostBuild.at(it.first).Equals(it.second))
-            return true;
+            return false;
     }
 
     for(const auto& it : Cleanup)
     {
         if(other.Cleanup.count(it.first) == 0 || !other.Cleanup.at(it.first).Equals(it.second))
-            return true;
+            return false;
     }
 
     return true;