Skip to content

Commit 17b856f

Browse files
authored
avoid array allocations for string split (#4895)
1 parent 4f96b1c commit 17b856f

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/ServiceControl.UnitTests/API/APIApprovals.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ string RemoveDataStoreSettings(string json)
150150

151151
foreach (var settingLine in settingsLines)
152152
{
153-
var parts = settingLine.Split(new[] { ':' }, StringSplitOptions.RemoveEmptyEntries);
153+
var parts = settingLine.Split(':', StringSplitOptions.RemoveEmptyEntries);
154154
var settingName = parts[0].Trim('"', ' ');
155155

156156
if (dataStoreSettings.Contains(settingName) == false)

src/ServiceControl/ExternalIntegrations/RemoveVersionInformationBehavior.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ public override Task Invoke(IOutgoingPhysicalMessageContext context, Func<Task>
1111
{
1212
if (context.Headers.TryGetValue(NServiceBus.Headers.EnclosedMessageTypes, out var typeHeader))
1313
{
14-
var types = typeHeader.Split(new[] { ';' }, StringSplitOptions.RemoveEmptyEntries);
14+
var types = typeHeader.Split(';', StringSplitOptions.RemoveEmptyEntries);
1515

1616
var typesWithoutVersionInfo = string.Join(";", types.Select(x => RemoveVersionAndKeyInformation(x)));
1717

0 commit comments

Comments
 (0)