Opened 15 years ago
Closed 15 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)
Change History (8)
Changed 15 years ago by
| Attachment: | 28c7ad6f8d91.patch added |
|---|
comment:1 follow-up: 2 Changed 15 years ago by
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 Changed 15 years ago by
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 follow-up: 4 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
[28c7ad6f8d91] has been merged to the main branch.
comment:4 follow-ups: 6 7 Changed 15 years ago by
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
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 15 years ago by
| Milestone: | → LEMON 1.2 release |
|---|
comment:6 Changed 15 years ago by
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 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
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.
Fix clear() function