ParadoxGameConverters / PGCG.commonItems / 23385239693

92%

Build Type

push

github

Committed by web-flow

Commit Message

Parsing optimizations and optional parameter in Parser constructor (#799)

* Parsing optimizations

Parser lexeme single-char tokens now reuse static cached strings (=, {, }, ?) instead of char.ToString allocations.
Parser.ParseStream token history string builder made lazy (only allocates when broken =/?= state detected) to reduce active allocations and speed up common flow.
Parser.ParseFolder path now uses Directory.EnumerateFiles, precomputed extension HashSet, and one loop, replacing List+RemoveWhere overhead.
Parser.ParseGameFolder path now uses extension HashSet + manual loop + parsed-target list - eliminating extra LINQ Select and intermediate memory.
Regex rule replacement path upgraded from O(n) linear string scan to dictionary index mapping (regexRuleIndices), making register/replace faster.
Comment-skipping in parser parsing (# lines) now uses BufferedReader.SkipRestOfLine() to avoid ReadLine string allocations.
Added SkipRestOfLine() to BufferedReader.
Added benchmark harness commonItems.Benchmarks tests for all optimization candidates and confirmed runtime/allocation improvements.
dotnet build and dotnet test both passed.

Includes validated performance:
Single-char token ~22% speedup and ~83% allocation reduction
ParseStream history ~56% speedup and ~93% allocation reduction
ParseGameFolder ~19% speedup and ~26% allocation reduction
ParseFolder ~6% speedup and ~11% allocation reduction
Regex replace ~86% speedup (with expected small dictionary cost)

* Avoid using a regexRuleIndices dictionary

* Avoid looping the parser initialization in LoadScriptValues

* Add a parameter to Parser constructor

* Bump version

Run Details

1279 of 1456 branches covered (87.84%)

Branch coverage included in aggregate %.

66 of 69 new or added lines in 4 files covered. (95.65%)

3 existing lines in 2 files now uncovered.

3795 of 4047 relevant lines covered (93.77%)

45530.72 hits per line