Opened 7 years ago

Closed 7 years ago

#108 closed defect (fixed)

Debug Trace not removed

Reported by: patrik.svestka@… Owned by:
Priority: minor Milestone: 8.0.0
Component: default Keywords:
Cc: Also affects CVS HEAD (eXept version): no

Description

After turning on Debug/Trace (from menu) and right afterwards switching on Debug/Trace Sender or Debug/Trace full walkback; It is impossible to remove the traces using menu Debug/Remove trace. Only Remove all Break- & Tracepoints works. (Maybe this was intentional?)

Change History (8)

comment:1 by jan vrany, 7 years ago

Milestone: → 8.0.0

comment:2 by jan vrany, 7 years ago

This is indeed a bug. It's caused by method being wrapped twice - the problem is probably in Tools::NewSystemBrowser >> commonTraceHelperWith:with:clear:

by jan vrany, 7 years ago

Attachment: Tools__NewSystemBrowserTests.st​ added

by jan vrany, 7 years ago

Attachment: libtool_fix_1_of_3_rev_c95ae929a0b1_Issue__108__Fixed_bug_causing_method_selection_loss_in_some_cases.patch​ added

by jan vrany, 7 years ago

Attachment: libtool_fix_2_of_3_rev_90d1d10f51ca_Issue__108__Fixed_update_of_method_list_in_case_methid_is_re_wrapped.patch​ added

by jan vrany, 7 years ago

Attachment: libtool_fix_3_of_3_rev_9ae4a3d621e0_Issue__108__Added_some_tests_for_issue__108.patch​ added

comment:3 by jan vrany, 7 years ago

Status: new → testing

Should be addressed in attached patches

comment:4 by patrik.svestka@…, 7 years ago

Resolution: → fixed
Status: testing → closed

The patches work perfectly. The traces are now correctly removed and tests do pass.

I'm closing this ticket.

Note: See TracTickets for help on using tickets.