This is the mail archive of the gdb-patches@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: [PATCH] add -s option to make -break-insert support dprintf


>>>>> "Hui" == Hui Zhu <teawater@gmail.com> writes:

Hui> Does GDB have some function can convert string to "" inside string?

I wouldn't rule it out but I couldn't think of one offhand.

Hui> Could you give me some comments about the new format for -s.
Hui> My thought is:
Hui> -s format argv1 argv2, other options
Hui> "-break-insert -s FORMAT LOCATION ARG ARG ARG ..." is too different
Hui> with current options of  -break-insert and very hard to support.

What is hard about it?

In mi_cmd_break_insert, if '-s' was given, use argv[oind+1] through
argv[argc-1] as the arguments.  Maybe the hard part is getting this
through create_breakpoint?  That is just further proof that it is a bad
API.

TBH I don't see why this needs to be in -break-insert.  A new command
seems cleaner to me.  But I don't want to press too hard either.

Tom


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