Faculty of Information Technology
Software Engineering Group

Opened 6 years ago

Closed 5 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 Changed 6 years ago by Jan Vrany

Milestone: → 8.0.0

comment:2 Changed 6 years ago by Jan Vrany

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

Changed 6 years ago by Jan Vrany

Attachment: Tools__NewSystemBrowserTests.st​ added

Changed 5 years ago by Jan Vrany

Attachment: libtool_fix_1_of_3_rev_c95ae929a0b1_Issue__108__Fixed_bug_causing_method_selection_loss_in_some_cases.patch​ added

Changed 5 years ago by Jan Vrany

Attachment: libtool_fix_2_of_3_rev_90d1d10f51ca_Issue__108__Fixed_update_of_method_list_in_case_methid_is_re_wrapped.patch​ added

Changed 5 years ago by Jan Vrany

Attachment: libtool_fix_3_of_3_rev_9ae4a3d621e0_Issue__108__Added_some_tests_for_issue__108.patch​ added

comment:3 Changed 5 years ago by Jan Vrany

Status: new → testing

Should be addressed in attached patches

comment:4 Changed 5 years ago by Patrik Svestka

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.