doc: clarify error handling in net.createServer

dirceu · evanlucas · commit 581606a5fdcd · 2016-02-25T05:57:14.000-06:00
Remove indications that an error argument is sent to the `listen()` callback. Fixes: #5345 PR-URL: #5353 Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Trevor Norris <trev.norris@gmail.com>
diff --git a/doc/api/net.markdown b/doc/api/net.markdown
@@ -48,11 +48,13 @@ Example:
 ```js
 var server = net.createServer((socket) => {
   socket.end('goodbye\n');
+}).on('error', (err) => {
+  // handle errors here
+  throw err;
 });
 
 // grab a random port.
-server.listen((err) => {
-  if (err) throw err;
+server.listen(() => {
   address = server.address();
   console.log('opened server on %j', address);
 });
@@ -660,9 +662,10 @@ const server = net.createServer((c) => {
   c.write('hello\r\n');
   c.pipe(c);
 });
-server.listen(8124, (err) => {
-  // 'listening' listener
-  if (err) throw err;
+server.on('error', (err) => {
+  throw err;
+});
+server.listen(8124, () => {
   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
 
 ```js
-server.listen('/tmp/echo.sock', (err) => {
-  // 'listening' listener
-  if (err) throw err;
+server.listen('/tmp/echo.sock', () => {
+  console.log('server bound');
 });
 ```
 
