This is the mail archive of the gdb@sourceware.org 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: [MI] argv/argc/args


On Wednesday 25 June 2008 21:21:07 Marc Khouzam wrote:
> Hi,
> 
> just to give input on DSF's use of these commands,
> although that doesn't speak for any other frontend.
> 
> > 	-exec-next
> > 	-exec-next-instruction
> > 	-exec-step
> > 	-exec-step-instruction
> > 	-exec-continue
> 
> DSF does use a parameter on some of these but it is only an integer.
> So, no problem here.
> 
>  
> > The bad ones are:
> > 
> > 	-exec-until
> 
> We use a parameter of format file:line
> I guess it could be a problem if the file name had a space in it.
> 
> > 	-target-download
> 
> Not used yet.  But any future use will require to have some special
> code if we want to support older GDBs.
> 
> > 	-target-select
> > 
> 
> The parameters here are host:port or serialDevice, which I believe
> will not contain spaces.  Should be fine.

Well, it can also be "| some-program lots-of-parameters", so quoting
is an issue. (And Sourcery G++ does use that ;-)

> > So, we have 3 commands for which requiring the input to be 
> > quoted per MI rules
> > will cause issues; and fixing those issues will require 
> > changing other parts of
> > GDB to avoid parsing filenames, which is risky at this point. 
> > It appears, that
> > instead of reverting my original patch, we can just path 
> > those 3 commands
> > via CLI directly. Does the plan sound reasonable?
> 
> For DSF, it should mean no changes, so I like this solution.

Ok.

- Volodya


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