Skip to content

src: avoid redundant call to std::get_if<>()#64094

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
tniessen:src-avoid-repeated-getif-calls
Jun 25, 2026
Merged

src: avoid redundant call to std::get_if<>()#64094
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
tniessen:src-avoid-repeated-getif-calls

Conversation

@tniessen

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Tobias Nießen <tniessen@tnie.de>
@tniessen tniessen added the c++ Issues and PRs that require attention from people who are familiar with C++. label Jun 23, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/crypto

@nodejs-github-bot nodejs-github-bot added crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run. labels Jun 23, 2026
@panva panva added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 23, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 23, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@panva panva added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 24, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 25, 2026
@nodejs-github-bot nodejs-github-bot merged commit a811c34 into nodejs:main Jun 25, 2026
95 of 96 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in a811c34

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants