1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

doc: clarify error handling in net.createServer

Browse Source 
Remove indications that an error argument is sent to the `listen()` callback.

Fixes: https://github.com/nodejs/node/issues/5345
PR-URL: https://github.com/nodejs/node/pull/5353
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
This commit is contained in:
Dirceu Pereira Tiegs 2016-02-21 15:07:27 -03:00 committed by Evan Lucas
parent 45789027ac
commit 581606a5fd
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
doc/api/net.markdown
View File

@ -48,11 +48,13 @@ Example:
```js ```js
var server = net.createServer((socket) => { var server = net.createServer((socket) => {
socket.end('goodbye\n'); socket.end('goodbye\n');
}).on('error', (err) => {
// handle errors here
throw err;
}); });
// grab a random port. // grab a random port.
server.listen((err) => { server.listen(() => {
if (err) throw err;
address = server.address(); address = server.address();
console.log('opened server on %j', address); console.log('opened server on %j', address);
}); });
@ -660,9 +662,10 @@ const server = net.createServer((c) => {
c.write('hello\r\n'); c.write('hello\r\n');
c.pipe(c); c.pipe(c);
}); });
server.listen(8124, (err) => { server.on('error', (err) => {
// 'listening' listener throw err;
if (err) throw err; });
server.listen(8124, () => {
console.log('server bound'); console.log('server bound');
}); });
``` ```
@ -677,9 +680,8 @@ To listen on the socket `/tmp/echo.sock` the third line from the last would
just be changed to just be changed to
```js ```js
server.listen('/tmp/echo.sock', (err) => { server.listen('/tmp/echo.sock', () => {
// 'listening' listener console.log('server bound');
if (err) throw err;
}); });
``` ```