Skip to content

Conversation

@victorgomes
Copy link

The value string added in (#51289) can sometimes be empty. We check for it before accessing it.

@nodejs-github-botnodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. labels Feb 14, 2024
@Flarna
Copy link
Member

seems to be a duplicate of #51665

@mertcanaltinmertcanaltin added the duplicate Issues and PRs that are duplicates of other issues or PRs. label Feb 14, 2024
@victorgomes
Copy link
Author

seems to be a duplicate of #51665

Indeed.

@victorgomesvictorgomes deleted the fix-empty-string branch February 14, 2024 15:19
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.duplicateIssues and PRs that are duplicates of other issues or PRs.needs-ciPRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@victorgomes@Flarna@nodejs-github-bot@mertcanaltin