Home Original page

src,lib: reducing C++ calls of esm legacy main resolve by H4ad · Pull Request #48325 · nodejs/node

@nodejs-github-bot added c++

Issues and PRs that require attention from people who are familiar with C++.

esm

Issues and PRs related to the ECMAScript Modules implementation.

fs

Issues and PRs related to the fs subsystem / file system.

needs-ci

PRs that need a full CI run.

labels

Jun 3, 2023

@H4ad H4ad mentioned this pull request

Jun 3, 2023

RafaelGSS

RafaelGSS

anonrig

anonrig

debadree25

RafaelGSS

@H4ad H4ad marked this pull request as ready for review

June 5, 2023 02:30

RaisinTen

Ceres6 pushed a commit to Ceres6/node that referenced this pull request

Aug 14, 2023 
PR-URL: nodejs#48664
Refs: nodejs#48325
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>

Ceres6 pushed a commit to Ceres6/node that referenced this pull request

Aug 14, 2023 
Instead of many C++ calls, now we make only one C++ call
to return a enum number that represents the selected state.

PR-URL: nodejs#48325
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>

Ceres6 pushed a commit to Ceres6/node that referenced this pull request

Aug 14, 2023 
PR-URL: nodejs#48664
Refs: nodejs#48325
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>

ruyadorno pushed a commit that referenced this pull request

Sep 11, 2023 
PR-URL: #48664
Refs: #48325
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>

ruyadorno pushed a commit that referenced this pull request

Sep 13, 2023 
PR-URL: #48664
Refs: #48325
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>

H4ad added a commit to H4ad/node that referenced this pull request

Sep 14, 2023 
Instead of many C++ calls, now we make only one C++ call
to return a enum number that represents the selected state.

Backport-PR-URL: nodejs#48325

H4ad added a commit to H4ad/node that referenced this pull request

Sep 14, 2023

H4ad added a commit to H4ad/node that referenced this pull request

Sep 14, 2023 
Instead of many C++ calls, now we make only one C++ call
to return a enum number that represents the selected state.

Backport-PR-URL: nodejs#48325

H4ad added a commit to H4ad/node that referenced this pull request

Sep 14, 2023

H4ad added a commit to H4ad/node that referenced this pull request

Sep 14, 2023 
Instead of many C++ calls, now we make only one C++ call
to return a enum number that represents the selected state.

Backport-PR-URL: nodejs#48325

H4ad added a commit to H4ad/node that referenced this pull request

Sep 14, 2023

@H4ad H4ad mentioned this pull request

Sep 14, 2023

ruyadorno pushed a commit that referenced this pull request

Sep 17, 2023 
PR-URL: #48664
Refs: #48325
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>

H4ad added a commit to H4ad/node that referenced this pull request

Sep 18, 2023 
Instead of many C++ calls, now we make only one C++ call
to return a enum number that represents the selected state.

Backport-PR-URL: nodejs#48325

H4ad added a commit to H4ad/node that referenced this pull request

Sep 18, 2023

H4ad added a commit to H4ad/node that referenced this pull request

Sep 18, 2023

This was referenced

Feb 14, 2024