Apply suggestions from code review

skottmckay · edgchen1 · web-flow · commit 2e6800d94d38 · 2024-03-11T18:17:00.000+10:00
Co-authored-by: Edward Chen &lt;18449977+edgchen1@users.noreply.github.com&gt;
diff --git a/onnxruntime/core/providers/partitioning_utils.cc b/onnxruntime/core/providers/partitioning_utils.cc
@@ -245,8 +245,7 @@ std::vector<std::vector<const Node*>> CreateSupportedPartitionNodeGroups(
                       process_downstream_node(output);
                     });
     } else {
-      std::for_each(node.OutputNodesBegin(), node.OutputNodesEnd(),
-                    [&](const Node& output) { process_downstream_node(output); });
+      std::for_each(node.OutputNodesBegin(), node.OutputNodesEnd(), process_downstream_node);
     }
 
     ++num_nodes_processed;
diff --git a/onnxruntime/test/providers/partitioning_utils_test.cc b/onnxruntime/test/providers/partitioning_utils_test.cc
@@ -135,7 +135,7 @@ TEST(PartitioningUtilsTest, TestHandlingQDQNodeUnitWithNoQNodes) {
 }
 
 // TopK produces 2 outputs, one of which is used in a QDQ node group (Q of values output)
-// and the other (indices output) is not. A downstream node consumuing the indices output has an edge from the target
+// and the other (indices output) is not. A downstream node consuming the indices output has an edge from the target
 // node and not a Q node.
 // To process this correctly, the QDQ NodeUnit must return output edges for both the Q node/s of the values output,
 // and the downstream node (Cast in this case) of the indices output.

Original file line number	Diff line number	Diff line change
`@@ -245,8 +245,7 @@ std::vector<std::vector<const Node*>> CreateSupportedPartitionNodeGroups(`
`245`	`245`	`process_downstream_node(output);`
`246`	`246`	`});`
`247`	`247`	`} else {`
`248`		`- std::for_each(node.OutputNodesBegin(), node.OutputNodesEnd(),`
`249`		`- [&](const Node& output) { process_downstream_node(output); });`
	`248`	`+ std::for_each(node.OutputNodesBegin(), node.OutputNodesEnd(), process_downstream_node);`
`250`	`249`	`}`
`251`	`250`
`252`	`251`	`++num_nodes_processed;`
Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ TEST(PartitioningUtilsTest, TestHandlingQDQNodeUnitWithNoQNodes) {`
`135`	`135`	`}`
`136`	`136`
`137`	`137`	`// TopK produces 2 outputs, one of which is used in a QDQ node group (Q of values output)`
`138`		`-// and the other (indices output) is not. A downstream node consumuing the indices output has an edge from the target`
	`138`	`+// and the other (indices output) is not. A downstream node consuming the indices output has an edge from the target`
`139`	`139`	`// node and not a Q node.`
`140`	`140`	`// To process this correctly, the QDQ NodeUnit must return output edges for both the Q node/s of the values output,`
`141`	`141`	`// and the downstream node (Cast in this case) of the indices output.`