This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] -stack-select-frame


 > - You're changing the behavior of a command; do people use this
 > command?  Are you confident that they will handle the new output
 > gracefully?

I'm not aware of anyone using this command - I don't see how they currently
could really.  I think its less likely to break existing behaviour than this
change:

2005-05-17  Daniel Jacobowitz  <dan@codesourcery.com>
	    Dennis Brueni  <dennis@slickedit.com>

	* stack.c (print_frame): In MI mode, output a fullname attribute
	with the stack frame.

which must change the output for every frontend using MI.

I don't see how MI can evolve without changing its behaviour.  If it is
significantly different presumably a new level can be added as before.

 > - Please, let's not add new MI features without matching testcases.

I'll be happy to add testcases if there is a likelihood that my patch will be
approved.

 > > ! ^done,frame=@{level="2",addr="0x000107a4",func="foo",
 > > !   file="recursive2.c",fullname="/home/foo/bar/devo/myproject/recursive2.c",line=line="14"@},
 > 
 > The double line= is a typo, right?

I've just copied it from another part of the manual.  In the node
"GDB/MI Command Description Format" it says:

Manual>    Note the the line breaks shown in the examples are here only for
Manual> readability.  They don't appear in the real output.


Nick


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]