#47644: refactor: avoid a few unnecessary strings
Merged
Description of Change
Small cleanup that I came across while working on something else.
In a handful of cases, we're instantiating a std::string
when a cheaper std::string_view
will do. This PR fixes that.
Checklist
- PR description included and stakeholders cc'd
-
npm test
passes - PR release notes describe the change in a way relevant to app developers, and are capitalized, punctuated, and past tense.
Release Notes
Notes: none.
Backports
36-x-y
PendingWaiting for a manual backport
37-x-y
MergedPR Number
#47654Merged At
Jul 4, 2025, 3:58:46 AM
Released In
v37.2.1Release Date
Jul 9, 2025, 11:02:22 AM
38-x-y
MergedPR Number
#47655Merged At
Jul 4, 2025, 3:58:50 AM
Released In
v38.0.0-alpha.4Release Date
Jul 7, 2025, 8:32:17 AM
Semver Impact
Major
Breaking changes
Minor
New features
Patch
Bug fixes
None
Docs, tests, etc.
Semantic Versioning helps users understand the impact of updates:
- Major (X.y.z): Breaking changes that may require code modifications
- Minor (x.Y.z): New features that maintain backward compatibility
- Patch (x.y.Z): Bug fixes that don't change the API
- None: Changes that don't affect using facing parts of Electron