Skip to content

src: remove unused function GetName() in node_perf#49244

Merged
nodejs-github-bot merged 2 commits intonodejs:mainfrom
pluris:remove/unused_function
Aug 21, 2023
Merged

src: remove unused function GetName() in node_perf#49244
nodejs-github-bot merged 2 commits intonodejs:mainfrom
pluris:remove/unused_function

Conversation

@pluris
Copy link
Copy Markdown
Contributor

@pluris pluris commented Aug 19, 2023

I looked through the code, but couldn't find where GetName() is used.
Both test and build passed, I think it can be deleted.

Loading
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++. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants