Re: Fix committed for mi-syn-frames fails (was: Re: PATCH RFA: Fix MI stack frame output for synthetic frames)

Jason Molenda
2003-02-06T18:27:49Z

On Thu, Feb 06, 2003 at 01:21:00PM -0500, Andrew Cagney wrote:

>
> Pss, add a comment explaining why the code is written that way :-)
>

I don't think anyone looks to the testsuite source code for brevity
and logical expression, but I don't have any real preference.
Patch below committed.

2003-02-06 Jason Molenda (jason-cl@molenda.com)

* gdb.mi/mi-syn-frame.c (subroutine): Add a comment explaining
why the code is written that way.

Index: gdb.mi/mi-syn-frame.c
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.mi/mi-syn-frame.c,v
retrieving revision 1.2
diff -u -p -r1.2 mi-syn-frame.c
--- gdb.mi/mi-syn-frame.c 6 Feb 2003 06:27:01 -0000 1.2
+++ gdb.mi/mi-syn-frame.c 6 Feb 2003 18:25:01 -0000
@@ -43,6 +43,10 @@ handler (int sig)
subroutine (sig);
}

+/* The first statement in subroutine () is a place for a breakpoint.
+ Without it, the breakpoint is put on the while comparison and will
+ be hit at each iteration. */
+
void
subroutine (int in)
{