Hi all, I make a new patch that change "can-use-displaced-stepping" to "displaced-stepping" and change some part according to Eli's idea in doc. 2008-10-22 Hui Zhu Pedro Alves * infrun.c (can_use_displaced_stepping): Change type to const char pointer. (can_use_displaced_stepping_auto): New string. (can_use_displaced_stepping_on): New string. (can_use_displaced_stepping_off): New string. (can_use_displaced_stepping_enum): New array. (show_can_use_displaced_stepping): In auto mode, also show the current effect of the option. (use_displaced_stepping): Return non-zero if displaced stepping is auto, and can be used with GDBARCH, and in non-stop mode. Return non-zero if displaced stepping is on, and can be used with GDBARCH. Return zero otherwise. (_initialize_infrun): Make the "set displaced-stepping" command an enum command. Change its class to class_run. Place it in the top level set list. Extend help to describe the auto mode. 2008-10-22 Hui Zhu Pedro Alves * gdb.texinfo (displaced-stepping): Describe the auto mode setting, and say it's the default. This is now a mainstream setting instead of a maintenance setting. Thanks, Hui On Thu, Oct 16, 2008 at 10:59, Pedro Alves wrote: > On Thursday 16 October 2008 03:38:37, Daniel Jacobowitz wrote: >> On Thu, Oct 16, 2008 at 01:07:42AM +0100, Pedro Alves wrote: > >> > I think Joel's suggestion was a good one: By gaining an "auto" >> > setting, this is as much a maintainer command as >> > "set breakpoint always-inserted" is. So, I believe it's time we move >> > the command to the top level "set" command group. >> > >> > set can-use-displaced-stepping (auto|on|off) >> > >> > Eli, can I ask you to go over the help strings and the documentation in >> > the patch below? Do they look OK-ish? >> >> That's a pretty long name. Do you think there are any other useful >> stepping related settings? If so, maybe we should have a "set step" >> prefix. >> > > Hmm, not offhand. I'd have to think about it. > What would you call the setting then, if not "set step can-use-displaced-stepping" ? > > The name was actually modeled on "set can-use-hw-watchpoints" in the first place. > > -- > Pedro Alves >