Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: use base::NoDestructor instead of base::LazyInstance #40978

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented Jan 12, 2024

Backport of #40947

See that PR for details.

Notes: none

@trop trop bot requested a review from a team as a code owner January 12, 2024 13:51
@trop trop bot requested a review from ckerr January 12, 2024 13:51
@trop trop bot added 28-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Jan 12, 2024
trop bot and others added 7 commits January 17, 2024 12:08
Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
…nMessageProvider()

Co-authored-by: Charles Kerr <charles@charleskerr.com>
…se_spellchecker_download_url.patch

Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
…rmissionMessageProvider()

make sure instance is static

Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
@codebytere codebytere force-pushed the trop/28-x-y-bp-refactor-use-base-nodestructor-instead-of-base-lazyinstance-1705067443142 branch from 2ba0791 to 923cdf5 Compare January 17, 2024 11:08
@jkleinsc jkleinsc merged commit 0f8ebfb into 28-x-y Jan 17, 2024
13 checks passed
@jkleinsc jkleinsc deleted the trop/28-x-y-bp-refactor-use-base-nodestructor-instead-of-base-lazyinstance-1705067443142 branch January 17, 2024 17:38
Copy link

release-clerk bot commented Jan 17, 2024

No Release Notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
28-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants