Skip to content

Commit

Permalink
lib: clean up usage of threw
Browse files Browse the repository at this point in the history
Use try/catch to instead of threw.

PR-URL: nodejs/node#10534
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
  • Loading branch information
JacksonTian authored and addaleax committed Sep 5, 2017
1 parent 63213c2 commit 298110e
Showing 1 changed file with 6 additions and 12 deletions.
18 changes: 6 additions & 12 deletions lib/internal/bootstrap_node.js
Original file line number Diff line number Diff line change
Expand Up @@ -466,20 +466,14 @@
}

function tryGetCwd(path) {
var threw = true;
var cwd;
try {
cwd = process.cwd();
threw = false;
} finally {
if (threw) {
// getcwd(3) can fail if the current working directory has been deleted.
// Fall back to the directory name of the (absolute) executable path.
// It's not really correct but what are the alternatives?
return path.dirname(process.execPath);
}
return process.cwd();
} catch (ex) {
// getcwd(3) can fail if the current working directory has been deleted.
// Fall back to the directory name of the (absolute) executable path.
// It's not really correct but what are the alternatives?
return path.dirname(process.execPath);
}
return cwd;
}

function evalScript(name) {
Expand Down

0 comments on commit 298110e

Please sign in to comment.