Skip to content

Commit ad8bca0

Browse files
committed
More Log sanitizing. Serius, guys - we don't mangle with the upstream just because. KSP.log format is here for years!
1 parent a9b6d14 commit ad8bca0

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

Source/ModuleManager/MMPatchLoader.cs

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -350,9 +350,6 @@ private bool IsCacheUpToDate()
350350
private bool CheckFilesChange(UrlDir.UrlFile[] files, ConfigNode shaConfigNode)
351351
{
352352
bool noChange = true;
353-
StringBuilder changes = new StringBuilder();
354-
355-
changes.Append("Changes :\n");
356353

357354
for (int i = 0; i < files.Length; i++)
358355
{
@@ -364,28 +361,29 @@ private bool CheckFilesChange(UrlDir.UrlFile[] files, ConfigNode shaConfigNode)
364361

365362
if (fileSha == null || filesSha[files[i].url] != fileSha)
366363
{
367-
changes.Append("Changed : " + fileNode.GetValue("filename") + ".cfg\n");
364+
logger.Info("Changed : " + fileNode.GetValue("filename") + ".cfg\n");
368365
noChange = false;
369366
}
370367
}
368+
371369
for (int i = 0; i < files.Length; i++)
372370
{
373371
ConfigNode fileNode = GetFileNode(shaConfigNode, files[i].url);
374372

375373
if (fileNode == null)
376374
{
377-
changes.Append("Added : " + files[i].url + ".cfg\n");
375+
logger.Info("Added : " + files[i].url + ".cfg\n");
378376
noChange = false;
379377
}
380378
shaConfigNode.RemoveNode(fileNode);
381379
}
380+
382381
foreach (ConfigNode fileNode in shaConfigNode.GetNodes())
383382
{
384-
changes.Append("Deleted : " + fileNode.GetValue("filename") + ".cfg\n");
383+
logger.Info("Deleted : " + fileNode.GetValue("filename") + ".cfg\n");
385384
noChange = false;
386385
}
387-
if (!noChange)
388-
logger.Info(changes.ToString());
386+
389387
return noChange;
390388
}
391389

0 commit comments

Comments
 (0)