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


 > > -stack-info-frame hasn't been implemented yet (I've think we've been here
 > > before) but it would probably be quite easy to implement and I guess it
 > > could work like I've made -stack-select-frame without an argument work.
 > 
 > *snicker* that's what I get for reading the manual.  I assumed it was
 > implemented.
 > 
 > Maybe it is time to mark the unimplemented commands in the manual?

It _is_ documented as unimplemented in my copy.

   ...
 > If you're OK with only a literal "-stack-select-frame" providing the
 > frame information, how about implementing -stack-info-frame instead? 
 > The documentation for -stack-select-frame does not suggest the argument
 > is optional; we could make it mandatory.  It just seems cleaner to me
 > to have select be write-only and info be read-only.

The CLI command is both write-only and read-only.  Clearly there's no
 need to copy what might be existing bad practice and the command name
-stack-select-frame to provide frame information is unintuitive. I'll 
submit a patch to make -stack-info-frame do this.

Nick


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