Skip to content

feat(libflux): use LazyLock#5572

Merged
mhilton merged 1 commit intomasterfrom
mhilton/libflux-lazylock
Nov 24, 2025
Merged

feat(libflux): use LazyLock#5572
mhilton merged 1 commit intomasterfrom
mhilton/libflux-lazylock

Conversation

@mhilton
Copy link
Contributor

@mhilton mhilton commented Nov 24, 2025

The rust standard library has had LazyLock for a while now, use this instead of once_cell.

Part of #5570

Checklist

Dear Author 👋, the following checks should be completed (or explicitly dismissed) before merging.

  • ✏️ Write a PR description, regardless of triviality, to include the value of this PR
  • 🔗 Reference related issues
  • 🏃 Test cases are included to exercise the new code
  • 🧪 If new packages are being introduced to stdlib, link to Working Group discussion notes and ensure it lands under experimental/
  • 📖 If language features are changing, ensure docs/Spec.md has been updated

Dear Reviewer(s) 👋, you are responsible (among others) for ensuring the completeness and quality of the above before approval.

@mhilton mhilton requested a review from a team as a code owner November 24, 2025 08:48
Copy link
Member

@jdockerty jdockerty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

There's some doc-style changes which need to be made and swapping tousize::MAX instead, but otherwise this seems good.

The rust standard library has had LazyLock for a while now, use this
instead of once_cell.
@mhilton mhilton force-pushed the mhilton/libflux-lazylock branch from ad854b8 to a0c22cc Compare November 24, 2025 10:10
@mhilton mhilton merged commit 0587033 into master Nov 24, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants