Qiskit / qiskit / 13506447276

88%

Build Type

push

github

Committed by web-flow

Commit Message

Simplify return type of `DAGCircuit::control_flow_op_nodes` (#13892)

Technically this is a breaking change, not because of the type change
(we're returning a subset of what we previously did), but because the
docstring previously _explicitly_ said that `None` was used for the
empty list.  In practice, this is a bit of a footgun (as I found), and
inconsistent with the other `DAGCircuit` methods.  It's not clear that
it has a performance advantage; technically it can save a single Python
empty-list allocation (`None` is a singleton), but this cost is pretty
trivial.

Run Details

17 of 17 new or added lines in 3 files covered. (100.0%)

6 existing lines in 2 files now uncovered.

77974 of 88584 relevant lines covered (88.02%)

357955.46 hits per line