branches/kdevelop/3.4/languages/cpp/debugger

[Vladimir Prus]

On Sunday 15 January 2006 15:03, Robert Gruber wrote:
> SVN commit 498306 by rgruber:
>
> Removed s_viewLocals state from gdb debugger as discussed on mailinglist
> a few weeks ago.
> The decision whether to request locals and params is now done by the
> VariableTree and no longer by the controller.
> CCMAIL:kdevelop-devel at kdevelop.org
.......
> --- branches/kdevelop/3.4/languages/cpp/debugger/gdbcontroller.cpp #498305:498306
> @@ -429,7 +429,7 @@
>  // all other commands are disabled.
>  void GDBController::programNoApp(const QString &msg, bool msgBox)
>  {
> -    state_ = (s_appNotStarted|s_programExited|(state_&(s_viewLocals|s_shuttingDown)));
> +    state_ = (s_appNotStarted|s_programExited|s_shuttingDown);
>      destroyCmds();

For the record, this is a bug. The previous code would retain the existing value of the s_shuttingDown flag,
while the new code will always set that flag. As result, once an application existed, it can't be debugged again,
not even after stopping debugger.

OTOH, it looks like the flag is never set in the debugger, I'll see about removing it altogether.

- Volodya