From ffdc59e83504f5f4678142300534a8bf6078c89e Mon Sep 17 00:00:00 2001 From: Richard Webb Date: Sun, 3 May 2026 12:42:51 +0100 Subject: [PATCH] Change checkSourceLengthRule to use Span.IsWhiteSpace instead of String.Trim String.Trim can allocate a new string on each call, whereas Span.IsWhiteSpace() shouldn't allocate at all. --- .../Rules/Conventions/SourceLength/SourceLengthHelper.fs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/FSharpLint.Core/Rules/Conventions/SourceLength/SourceLengthHelper.fs b/src/FSharpLint.Core/Rules/Conventions/SourceLength/SourceLengthHelper.fs index 0099d4995..0eae2947d 100644 --- a/src/FSharpLint.Core/Rules/Conventions/SourceLength/SourceLengthHelper.fs +++ b/src/FSharpLint.Core/Rules/Conventions/SourceLength/SourceLengthHelper.fs @@ -69,7 +69,7 @@ let checkSourceLengthRule (config:Config) range fileContents errorName (skipRang let sourceCodeLines = sourceCode.Split([| '\n'; '\r' |]) let blankLinesCount = sourceCodeLines - |> Seq.filter (fun line -> line.Trim().Length = 0) + |> Seq.filter (fun line -> line.AsSpan().IsWhiteSpace()) |> Seq.length let skippedLinesCount =