perf: use OrdinalIgnoreCase Contains in HtmlReportGenerator span mapping by thomhurst · Pull Request #6093 · thomhurst/TUnit (original) (raw)

@thomhurst

Replace the ToLowerInvariant() copy + Contains chain in MapSpanService with in-place case-insensitive string.Contains(..., StringComparison. OrdinalIgnoreCase) calls, removing a per-span string allocation while preserving identical case-insensitive matching behavior.

The final return at line 765 keeps ToLowerInvariant() because its value is serialized as the span 'service' identifier consumed downstream as a CSS/URL token.

Closes #6057

[claude[bot]](/apps/claude)

@thomhurst thomhurst deleted the perf/issue-6057-htmlreport-ordinal branch

May 28, 2026 20:39

This was referenced

May 29, 2026

This was referenced

Jun 13, 2026

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})