Report cached errors in headers multiple times when included in multiple source files.

nirbar · nirbar · commit f172bc1a24ec · 2016-05-31T16:22:42.000+03:00
This somewhat reduces unique repoting, but follows full-scan convention.
diff --git a/lib/cache.cpp b/lib/cache.cpp
@@ -173,10 +173,15 @@ bool Cache::ReportCachedResults(const char* filePath, const char* configuration,
 	{
 		ErrorLogger::ErrorMessage msg;
 		string t = currChild->GetText();
-		if (msg.deserialize(t))
+		if (!msg.deserialize(t))
 		{
-			pLogger->reportErr(msg);
+			return false;
 		}
+		if (msg._id.compare("syntaxError") != 0)
+		{
+			msg.file0 = filePath;
+		}
+		pLogger->reportErr(msg);
 	}
 
 	return true;

Original file line number	Diff line number	Diff line change
`@@ -173,10 +173,15 @@ bool Cache::ReportCachedResults(const char* filePath, const char* configuration,`
`173`	`173`	`{`
`174`	`174`	`ErrorLogger::ErrorMessage msg;`
`175`	`175`	`string t = currChild->GetText();`
`176`		`- if (msg.deserialize(t))`
	`176`	`+ if (!msg.deserialize(t))`
`177`	`177`	`{`
`178`		`- pLogger->reportErr(msg);`
	`178`	`+ return false;`
`179`	`179`	`}`
	`180`	`+ if (msg._id.compare("syntaxError") != 0)`
	`181`	`+ {`
	`182`	`+ msg.file0 = filePath;`
	`183`	`+ }`
	`184`	`+ pLogger->reportErr(msg);`
`180`	`185`	`}`
`181`	`186`
`182`	`187`	`return true;`