Update ValueStringBuilder.EnsureCapacity.cs

[Aniobodo]

Changed Length >= newCapacity to Capacity >= newCapacity: checking with current capacity of the buffer.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[linkdotnet]

Oh that is a good one and much appreciated. @Joy-less I think that change Length >= newCapacity was unintentional?

Therefore the PR is justified. Given that we might resize too often at the moment.
@Aniobodo if you feel especially nice you can add an entry to the https://github.com/linkdotnet/StringBuilder/blob/main/CHANGELOG.md

Something under the UNRELEASED section:

## UNRELEASED

### CHANGED
- Optimized when the internal buffer should grow. Fixed by [@Aniobodo](https://github.com/Aniobodo)

[Joy-less]

Sorry yes this is correct, my bad if I introduced this mistake. @linkdotnet
Easy to get confused between the two, since Length isn't actually the length of the buffer.

[linkdotnet]

Sorry yes this is correct, my bad if I introduced this mistake. @linkdotnet Easy to get confused between the two, since Length isn't actually the length of the buffer.

Easy - no need to apologize. I also didn't spot that. Just wanted to make sure that it wasn't intentional. Let's get that in - I will add the changelog entry.