Community
Participate
Working Groups
Build Identifier: I like the feature of Eclipse and DSF that if the user steps while debugging the debug view doesn’t update for half a second with an anticipation that the target will be suspended again. The problem is that under some conditions (slow connection to the target) stepping takes around a second and the debug view flickers badly. I noticed that half second is a hard coded delay in DSF’s class SteppingController. Is it possible for a DSF debuggers to be able to customize the stepping controller somehow? One way is to let the user customize that with a preference option. Another way is for the Stepping Controller to be customizable from the DSF debugger. A third way is the Stepping Controller to gather some kind of stepping statistics and automatically readjust the delay within a reasonable interval based on date collected. Reproducible: Always Steps to Reproduce: NA
In our debugger we handle this case by providing cached frame information as long as a step is in progress, instead of returning an error because the target is running. This avoids the flicker (collapsing of the thread node). But, it's also no problem to make the delay customizable for debugger implementations. A simple setStepTimeout(int) should do, I suppose?
(In reply to comment #1) > In our debugger we handle this case by providing cached frame information as > long as a step is in progress, instead of returning an error because the target > is running. This avoids the flicker (collapsing of the thread node). > But, it's also no problem to make the delay customizable for debugger > implementations. A simple setStepTimeout(int) should do, I suppose? Adding public setStepTimeout(int) to the class SteppingController will work for me. May be it should be pared with getStepTimeout for completeness.
Created attachment 177814 [details] Patch for review This is what I intend to commit. Please review.
The proposed change satisfies my request. Thanks you.
Committed to HEAD.
Tony, can we have this change in CDT 7.0.x as well?
SteppingController is provisional API, so I think this should be OK.
Committed also to cdt_7_0.
