Re: [Changeset] Re: Q: Low hanging fruit?

[David Bateman]

David Bateman wrote:

> Ok then what about the attached instead. It moves
> 
> * dbcont, dbnext, dbstep functions to DEFCMD's
> * adds the dbquit function
> * modifies the dbstep command to be compatible
> * splits get_user_input into two versions, with the debug version using
> the standard parser and thus allowing multi-line input and things like
> "x = 1; return" in the debug mode to work.
> * Makes the quit, exit commands at the debug prompt quit Octave in the
> same manner as in Matlab.
> * Updates the debug.txi file.
> 
> Making the debug mode accept multi-line input was much easier than I
> thought it would be. I've tested this a little, but a little more
> testing might be a good thing.
>
Please also apply the attached trivial patch after applying the original
patch. It gets rid of a temporary external variable I introduced but
didn't use in the final version.

D.

# HG changeset patch
# User David Bateman <address@hidden>
# Date 1210712517 -7200
# Node ID 80c6707d0a901822ce4396d006cb35c57714de82
# Parent  c695eae655efeda36ea8cc864f5391cef4eb1f33
Remove unused variable from previous debug patch

diff --git a/src/input.h b/src/input.h
--- a/src/input.h
+++ b/src/input.h
@@ -91,9 +91,6 @@ extern OCTINTERP_API bool Vdrawnow_reque
 // TRUE if we are in debugging mode.
 extern bool Vdebugging;
 
-// TRUE if we are returning from debugging mode.
-extern bool Vdebugging_return;
-
 // The current line that we are debugging
 extern int Vdebugging_current_line;