Commit 520369d
committed
doc: fix exec example in child_process
Previously, the example was checking for error by strict equality to
null. The error could be undefined though which would fail that check.
PR-URL: #6660
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Jeremy Whitlock <jwhitlock@apache.org>1 parent 51d1960 commit 520369d
1 file changed
Lines changed: 7 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -140,13 +140,13 @@ generated output. | |||
| 140 | 140 | ||
| 141 | 141 | ```js | |
| 142 | 142 | const exec = require('child_process').exec; | |
| 143 | - const child = exec('cat *.js bad_file | wc -l', | ||
| 144 | - (error, stdout, stderr) => { | ||
| 145 | - console.log(`stdout: ${stdout}`); | ||
| 146 | - console.log(`stderr: ${stderr}`); | ||
| 147 | - if (error !== null) { | ||
| 148 | - console.log(`exec error: ${error}`); | ||
| 149 | - } | ||
| 143 | + exec('cat *.js bad_file | wc -l', (error, stdout, stderr) => { | ||
| 144 | + if (error) { | ||
| 145 | + console.error(`exec error: ${error}`); | ||
| 146 | + return; | ||
| 147 | + } | ||
| 148 | + console.log(`stdout: ${stdout}`); | ||
| 149 | + console.log(`stderr: ${stderr}`); | ||
| 150 | 150 | }); | |
| 151 | 151 | ``` | |
| 152 | 152 | ||
0 commit comments