polish

Author: panyx0718

paddle/fluid/framework/details/var_handle.h

@@ -56,7 +56,8 @@ struct VarHandleBase {
 
   void RemoveOutput(OpHandleBase* out, ir::Node* node) {
     pending_ops_.erase(out);
-    std::remove(node_->outputs.begin(), node_->outputs.end(), node);
+    node_->outputs.erase(
+        std::remove(node_->outputs.begin(), node_->outputs.end(), node));
   }
 
   void ClearGeneratedOp() {

paddle/fluid/framework/ir/graph.h

@@ -64,9 +64,6 @@ class Graph {
     return nodes.back().get();
   }
 
-  // TODO(paddle-dev): There shouldn't be kNone nodes in the ir::Graph.
-  // node should either be a executable kOperation or a kVariable. kNone
-  // node is a temporary solution.
   ir::Node* CreateEmptyNode(const std::string& name, ir::Node::Type type) {
     nodes.emplace_back(new ir::Node(name, type));
     return nodes.back().get();