codebytere

#53071: chore: fix clang-tidy dead-store warning in browser_win.cc

Merged
Created: Aug 21, 2026, 4:49:10 AM
Merged: Aug 21, 2026, 6:00:27 AM
4 comments
Target: main

Description of Change

windows-x64 / clang-tidy is currently red on every open PR with clang-analyzer-deadcode.DeadStores at shell/browser/browser_win.cc:233: RegQueryValueEx was passed &(size = sizeof(startup_binary)), and the analyzer cannot see the value being read back through the pointer. Initialize size on its own line and pass &size; behaviour is identical.

Checklist

Release Notes

Notes: none

Backports

42-x-y
Merged
PR Number
#53084
Merged At
Aug 21, 2026, 11:13:13 AM
Released In
Not yet
Release Date
Not yet
43-x-y
Merged
PR Number
#53086
Merged At
Aug 21, 2026, 11:11:52 AM
Released In
Not yet
Release Date
Not yet
44-x-y
Merged
PR Number
#53085
Merged At
Aug 21, 2026, 11:12:16 AM
Released In
Not yet
Release Date
Not yet

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