COIN-OR::LEMON - Graph Library

Opened 9 years ago

Closed 9 years ago

#335 closed defect (fixed)

Bug in ExtendFindEnum::clear

Reported by: Balazs Dezso Owned by: Alpar Juttner
Priority: major Milestone: LEMON 1.2 release
Component: core Version: hg main
Keywords: Cc:
Revision id:

Description

The clear function is not called properly.

Attachments (1)

28c7ad6f8d91.patch (622 bytes) - added by Balazs Dezso 9 years ago.
Fix clear() function

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by Balazs Dezso

Attachment: 28c7ad6f8d91.patch added

Fix clear() function

comment:1 in reply to:  description ; Changed 9 years ago by Alpar Juttner

Replying to deba:

The clear function is not called properly.

How did you find this bug?

At least, it shows that our test coverage is far from being full.

comment:2 in reply to:  1 Changed 9 years ago by Balazs Dezso

Replying to alpar:

Replying to deba:

The clear function is not called properly.

How did you find this bug?

At least, it shows that our test coverage is far from being full.

I tried to find a bug, which caused segfault in my program, and I checked this class line-by-line. The other bug did not exist, the segfault was caused by stack overflow, because the extractBlossom() in weighted matching uses recursive call. Indeed, it may have to be changed to iterative implementation sometime.

comment:3 Changed 9 years ago by Peter Kovacs

Resolution: fixed
Status: newclosed

[28c7ad6f8d91] has been merged to the main branch.

comment:4 in reply to:  3 ; Changed 9 years ago by Peter Kovacs

Resolution: fixed
Status: closedreopened

Replying to kpeter:

[28c7ad6f8d91] has been merged to the main branch.

Why don't we merged this bugfix to the 1.0 and 1.1 branches?

comment:5 Changed 9 years ago by Peter Kovacs

Milestone: LEMON 1.2 release

comment:6 in reply to:  4 Changed 9 years ago by Peter Kovacs

Replying to kpeter:

Why don't we merged this bugfix to the 1.0 and 1.1 branches?

Correctly: Why didn't we merge this bugfix to the branches 1.0 and 1.1?

comment:7 in reply to:  4 Changed 9 years ago by Alpar Juttner

Resolution: fixed
Status: reopenedclosed

Replying to kpeter:

Replying to kpeter:

[28c7ad6f8d91] has been merged to the main branch.

Why don't we merged this bugfix to the 1.0 and 1.1 branches?

Good point. See [cf0c1b85618c] and [268a052c3043] for the backport.

Note: See TracTickets for help on using tickets.