module: simplify tryStatSync with throwIfNoEntry option

aduh95 · ruyadorno · commit 543ecbdbd83e · 2021-01-21T17:57:37.000-05:00
PR-URL: #36971 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/lib/internal/modules/esm/resolve.js b/lib/internal/modules/esm/resolve.js
@@ -104,13 +104,8 @@ function getConditionsSet(conditions) {
 const realpathCache = new SafeMap();
 const packageJSONCache = new SafeMap(); /* string -> PackageConfig */
 
-function tryStatSync(path) {
- try {
- return statSync(path);
- } catch {
- return new Stats();
- }
-}
+const tryStatSync =
+ (path) => statSync(path, { throwIfNoEntry: false }) ?? new Stats();
 
 function getPackageConfig(path, specifier, base) {
 const existing = packageJSONCache.get(path);

-Original file line number
+Diff line change
 constrealpathCache=newSafeMap();
 constpackageJSONCache=newSafeMap();/* string -> PackageConfig */
 -functiontryStatSync(path){
 -try{
 -returnstatSync(path);
 -}catch{
 -returnnewStats();
 -}
 -}
 +consttryStatSync=
 +(path)=>statSync(path,{throwIfNoEntry: false})??newStats();
 functiongetPackageConfig(path,specifier,base){
 constexisting=packageJSONCache.get(path);