Skip to content

Commit f9b61ee

Browse files
committed
Fix logic bug in CollapseMessagesToSubfolder()
1 parent fc408f1 commit f9b61ee

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

src/StructuredLogger/Analyzers/BuildAnalyzer.cs

+8-6
Original file line numberDiff line numberDiff line change
@@ -378,14 +378,16 @@ private void AnalyzeTask(Task task)
378378
private void CollapseMessagesToSubfolder(Task task)
379379
{
380380
var messages = task.Children.OfType<Message>().ToArray();
381-
if (messages.Length > 10)
381+
if (messages.Length <= 10)
382382
{
383-
for (int i = task.Children.Count - 1; i >= 0; i--)
383+
return;
384+
}
385+
386+
for (int i = task.Children.Count - 1; i >= 0; i--)
387+
{
388+
if (task.Children[i] is Message)
384389
{
385-
if (task.Children[i] is Message)
386-
{
387-
task.Children.RemoveAt(i);
388-
}
390+
task.Children.RemoveAt(i);
389391
}
390392
}
391393

0 commit comments

Comments
 (0)