refactored com.mongodb.DBTCPConnector#_checkWriteError to make intent clearer.

Author: jyemin

src/main/com/mongodb/DBTCPConnector.java

@@ -123,16 +123,12 @@ void _checkClosed(){
             throw new IllegalStateException( "this Mongo has been closed" );
     }
 
-    WriteResult _checkWriteError( DB db , MyPort mp , DBPort port , WriteConcern concern )
+    WriteResult _checkWriteError( DB db, DBPort port , WriteConcern concern )
         throws MongoException, IOException {
-        CommandResult e = null;
-        e = port.runCommand( db , concern.getCommand() );
-
-        if ( ! e.hasErr() )
-            return new WriteResult( e , concern );
+        CommandResult e = port.runCommand( db , concern.getCommand() );
 
         e.throwOnError();
-        return null;
+        return new WriteResult( e , concern );
     }
 
     @Override
@@ -155,7 +151,7 @@ public WriteResult say( DB db , OutMessage m , WriteConcern concern , ServerAddr
             port.checkAuth( db );
             port.say( m );
             if ( concern.callGetLastError() ){
-                return _checkWriteError( db , mp , port , concern );
+                return _checkWriteError( db , port , concern );
             }
             else {
                 return new WriteResult( db , port , concern );