From 284f612965ed1bbf115acff94921af4f5f16db96 Mon Sep 17 00:00:00 2001 From: "Michael B. Gale" Date: Fri, 14 Mar 2025 14:06:48 +0000 Subject: [PATCH] C#: Use `StringBuilder` for feed arguments in `GetRestoreArgs` --- .../Semmle.Extraction.CSharp.DependencyFetching/DotNet.cs | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/DotNet.cs b/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/DotNet.cs index cf9f7f4105d..36c85cba9e2 100644 --- a/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/DotNet.cs +++ b/csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/DotNet.cs @@ -2,7 +2,7 @@ using System.Collections.Generic; using System.IO; using System.Linq; - +using System.Text; using Newtonsoft.Json.Linq; using Semmle.Util; @@ -71,10 +71,13 @@ namespace Semmle.Extraction.CSharp.DependencyFetching // the configuration file(s). if (restoreSettings.Sources != null) { + var feedArgs = new StringBuilder(); foreach (string source in restoreSettings.Sources) { - args += $" -s {source}"; + feedArgs.Append($" -s {source}"); } + + args += feedArgs.ToString(); } if (restoreSettings.ForceReevaluation)