ckerr

#47644: refactor: avoid a few unnecessary strings

Merged
Created: Jul 2, 2025, 12:51:44 PM
Merged: Jul 3, 2025, 11:08:59 AM
4 comments
Target: main

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

Release Notes

Notes: none.

Backports

36-x-y
Pending
Waiting for a manual backport
37-x-y
Merged
PR Number
#47654
Merged At
Jul 4, 2025, 3:58:46 AM
Released In
v37.2.1
Release Date
Jul 9, 2025, 11:02:22 AM
38-x-y
Merged
PR Number
#47655
Merged At
Jul 4, 2025, 3:58:50 AM
Released In
v38.0.0-alpha.4
Release 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