jpedroh / generic-merge / 10120435053

80%

push

github

perf: Use Cow for value in MergedCSTNode (#64)

In most scenarios, we don't need to create a new `String` when merging
the values of two CSTNodes. More often than not, we simply pick the
value from either the left or right node, or, ideally, the values of
both nodes are equal to the base. Consequently, creating owned Strings
is uncommon, and it would be more efficient to return a reference to the
value of one of the `CSTNodes`.

To accommodate this use case, this PR updates the `MergedCSTNode` to use
`Cow<str>` instead of `String`. This change eliminates unnecessary
`to_string` calls, potentially leading to performance improvements in
some scenarios.

576 of 1044 branches covered (55.17%)

Branch coverage included in aggregate %.

108 of 110 new or added lines in 5 files covered. (98.18%)

2054 of 2214 relevant lines covered (92.77%)

286315.6 hits per line