fixes and comments

Author: nbbeeken

src/change_stream.ts

@@ -897,7 +897,7 @@ export class ChangeStream<
       this.cursor.close().then(
         () => null,
         () => null
-      ); // TODO(NODE-XXX): Is this correct?
+      ); // Ignoring the result of close is intentional
 
       const topology = getTopology(this.parent);
       topology.selectServer(this.cursor.readPreference, {}, serverSelectionError => {
@@ -930,7 +930,7 @@ export class ChangeStream<
       this.cursor.close().then(
         () => null,
         () => null
-      ); // TODO(NODE-XXX): Is this correct?;
+      ); // Ignoring the result of close is intentional
 
       const topology = getTopology(this.parent);
       topology.selectServer(this.cursor.readPreference, {}, serverSelectionError => {

src/cmap/connect.ts

@@ -490,11 +490,7 @@ function makeSocks5Connection(options: MakeConnectionOptions, callback: Callback
             callback
           );
         },
-        error => {
-          if (error) {
-            return callback(connectionFailureError('error', error));
-          }
-        }
+        error => callback(connectionFailureError('error', error))
       );
     }
   );

src/cursor/abstract_cursor.ts

@@ -886,7 +886,7 @@ class ReadableCursorStream extends Readable {
           this._cursor.close().then(
             () => null,
             () => null
-          ); // TODO(NODE-XXXX): is this correct?
+          ); // Ignoring the result of close is intentional
           return this.push(null);
         }
 
@@ -908,7 +908,7 @@ class ReadableCursorStream extends Readable {
         this._cursor.close().then(
           () => null,
           () => null
-        ); // TODO(NODE-XXXX): is this correct?
+        ); // Ignoring the result of close is intentional
       } else {
         if (this.push(result)) {
           return this._readNext();

src/operations/execute_operation.ts

@@ -147,7 +147,7 @@ export function executeOperation<
         session.endSession().then(
           () => null,
           () => null
-        );
+        ); // Ignoring the result of endSession is intentional
       }
 
       throw error;