Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: fix/issue-13610-stabilizer-backend
DEFAULT BRANCH: main
Repo Added
Files 875
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH gh-readonly-queue/stable/2.1/pr-14959-20ba77d26d69524586fa336c5833e3b34995b898
branch: gh-readonly-queue/stable/2.1/pr-14959-20ba77d26d69524586fa336c5833e3b34995b898
CHANGE BRANCH
x

coverage: 87.738% (-0.008%) from 87.746%
17434703469

push

github

web-flow 
Fix: Memory leakage in `QkCountOps` creation. (backport #14930) (#14959)

* Fix: Memory leakage in `QkCountOps` creation. (#14930)

* Fix: Memory leakage in `QkCountOps` creation.
The following commits attempt to fix some memoery issues that the struct `QkOpCounts`. The `qk_opcounts_free` method would segfault if an empty object was passed. The function now checks whether the passed option includes a null pointer or an empty instance. The commits also change how the `data` attribute is generated without using `std::mem::forget`, but instead collection the `OpCount` vec into a boxed_slice and consuming it into a `Box`. Inspired by what was done for #14884.

* Docs: Add release note.

* Fix: Ironic memory leakage

* Fix c opcounts free suggestion (#7)

* Rename to _clear, free name

* rm dangling file

* use pointers

* ... forgot to register test

* fix `size_t`

* Fix: Missed some renaming

* add note on clearing opscount

* be explicit about loading Box<[OpCount]>

* fix merge artifact

* Apply suggestions from code review

Co-authored-by: Eli Arbel <46826214+eliarbel@users.noreply.github.com>

* goto

* happy miri happy life

* Fix merge artifacts

---------

Co-authored-by: Julien Gacon <gaconju@gmail.com>
Co-authored-by: Julien Gacon <jules.gacon@googlemail.com>
Co-authored-by: Eli Arbel <46826214+eliarbel@users.noreply.github.com>
(cherry picked from commit 28aa4bdaa)

# Conflicts:
#	test/c/test_elide_permutations.c
#	test/c/test_optimize_1q_sequences.c
#	test/c/test_sabre_layout.c
#	test/c/test_split_2q_unitaries.c

* rm merge artifacts

---------

Co-authored-by: Raynel Sanchez <87539502+raynelfss@users.noreply.github.com>
Co-authored-by: Julien Gacon <jules.gacon@googlemail.com>

23 of 23 new or added lines in 1 file covered. (100.0%)

19 existing lines in 4 files now uncovered.

80030 of 91215 relevant lines covered (87.74%)

511255.11 hits per line

Relevant lines Covered
91215 RELEVANT LINES 80030 COVERED LINES
511255.11 HITS PER LINE
Source Files on gh-readonly-queue/stable/2.1/pr-14959-20ba77d26d69524586fa336c5833e3b34995b898

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
17434703469 gh-readonly-queue/stable/2.1/pr-14959-20ba77d26d69524586fa336c5833e3b34995b898 Fix: Memory leakage in `QkCountOps` creation. (backport #14930) (#14959) * Fix: Memory leakage in `QkCountOps` creation. (#14930) * Fix: Memory leakage in `QkCountOps` creation. The following commits attempt to fix some memoery issues that the s... push web-flow github
87.74
See All Builds (14455)