Patrick Monnerat [Sat, 30 Jul 2016 15:36:42 +0000 (16:36 +0100)]
Tcl 8.6 compatibility.
Version 8.6 of Tcl changes some procedure parameters to const. Hopefully,
a CONST86 macro is defined by Tcl for this purpose. Use it where necessary.
* gdbtk/generic/gdbtk.c: define CONST84 and CONST86 macros as empty by default.
(gdbtk_notifier_set_timer): define parameter with CONST86.
(gdbtk_notifier_wait_for_event): likewise.
(gdbtk_notifier_procs): cast initializers.
Patrick Monnerat [Wed, 29 Jun 2016 15:08:50 +0000 (17:08 +0200)]
Upgrade submodule. Modify patches and code accordingly.
The upgrade introduces a new interpreter API. The main loop is now under
gdb control.
The input stream handling has also changed: everything goes through a
struct ui. To support it, a new gdb patch has been introduced.
* patches/binutils-gdb/007-Avoid-creating-a-file-handler-for-a-negative-fd.patch
New patch.
* gdbtk/generic/gdbtk-hooks.c (gdbtk_warning): add new parameter to
stderr_fileopen.
* gdbtk/generic/gdbtk-interp.c (gdbtk_command_loop): deleted.
(gdbtk_supports_command_editing): new dummy procedure.
(gdbtk_pre_command_loop): new procedure. Sets input fd to -1 to disable
gdb commands from standard input.
(gdbtk_interp_factory): new procedure.
(_initialize_gdbtk_interp): use new interpreter API.
Patrick Monnerat [Tue, 28 Jun 2016 15:48:59 +0000 (17:48 +0200)]
Insert Tcl event loop into gdb event loop. Use gdb event loop.
This is done via a new Tcl notifier and a gdb asynchronous event to call
Tcl_DoOneEvent when needed.
Gdb event handler needs some updates to support the Tcl notifier: they are
currently implemented by patches.
* patches/binutils-gdb/005-gdb_do_one_event-add-an-expiration-parameter.patch:
New patch.
* patches/binutils-gdb/006-add_file_handler-implement-a-mask-parameter.patch
New patch.
* gdbtk/generic/gdbtk.c (gdbtk_notifier_schedule_proc): New procedure.
(gdbtk_notifier_reschedule_tcl): New procedure.
(gdbtk_notifier_get_file_data): New procedure.
(gdbtk_notifier_file_handler_event_proc): New procedure.
(gdbtk_notifier_file_proc): New procedure.
(gdbtk_notifier_timeout): New procedure.
(gdbtk_notifier_set_timer): New procedure.
(gdbtk_notifier_wait_for_event): New procedure.
(gdbtk_notifier_delete_file_handler): New procedure.
(gdbtk_notifier_create_file_handler): New procedure.
(gdbtk_notifier_initialize): New procedure.
(gdbtk_notifier_finalize): New procedure.
(gdbtk_notifier_alert): New procedure.
(gdbtk_notifier_service_mode_hook): New procedure.
(gdbtk_install_notifier): New procedure.
(gdbtk_uninstall_notifier): New procedure.
(cleanup_init): Uninstall notifier.
(gdbtk_init): Install notifier.
* gdbtk/generic/gdbtk-hooks.c (gdbtk_call_command): Add timeout parameter to
gdb_do_one_event.
* gdbtk/generic/gdbtk-interp.c (gdbtk_command_loop): Use gdb start_event_loop
instead of Tk_MainLoop.
Patrick Monnerat [Fri, 22 Apr 2016 13:13:13 +0000 (15:13 +0200)]
Adjust C code for C++ syntax compatibility.
Gdb will soon be compiled as C++ code, thus our source code must match
this requirement.
* gdbtk/generic/gdbtk-bp.c (Gdbtk_Breakpoint_Init): Explicit casts to
ClientData in Tcl_CreateObjCommand() calls.
(tracepoint_exists): Explicit cast xmalloc() result.
* gdbtk/generic/gdbtk-cmds.c (Gdbtk_Init): Explicit casts to ClientData in
Tcl_CreateObjCommand() calls.
(wrapped_call): Explicit cast to ClientData of first callback call argument.
(do_listfiles): Explicit cast for initial info assignment.
Explicit cast of xrealloc() result.
(gdb_search): `space' is an enum search_domain, not a domain_enum.
(gdb_loadfile): Explicit cast 2nd catch_errors() argument to void *.
(perror_with_name_wrapper): Explicit cast of perror_with_name() argument.
* gdbtk/generic/gdbtk-hooks.c (gdbtk_two_elem_cmd): Explicit cast of
malloc() result.
(gdbtk_ignorable_warning): Rename formal argument 'class', as it is a
C++ reserved word.
* gdbtk/generic/gdbtk-interp.c (gdbtk_interpreter_ui_out): Explicit cast
initializer value of `data'.
* gdbtk/generic/gdbtk-register.c (Gdbtk_Register_Init): Explicit casts to
ClientData in Tcl_CreateObjCommand() calls.
(setup_architecture_data): Explicit cast of xcalloc() result.
* gdbtk/generic/gdbtk-stack.c (Gdbtk_Stack_Init): Explicit casts to ClientData
in Tcl_CreateObjCommand() calls.
* gdbtk/generic/gdbtk-varobj.c (variable_create): Variable `how_specified' is
an enum varobj_type, not an int.
* gdbtk/generic/gdbtk-wrapper.c (GDB_varobj_update): Rename argument
`explicit', as this is a C++ reserved word.
Properly cast `*changes' assignment value.
(wrap_type_print): Explicit cast of varstring initializer.
(wrap_varobj_update): Rename variable `explicit', as this is a C++ reserved
word.
* gdbtk/generic/gdbtk-wrapper.h (GDB_varobj_update): Rename argument
`explicit', as this is a C++ reserved word.
* gdbtk/generic/gdbtk.c: Unconditionally declare external Tktable_Init() as C
procedure.
* gdbtk/generic/gdbtk.h: Do not (wrongly) redeclare ptid_t.
* libgui/src/tkTable.h: Allow inclusion from a C++ source.
* libgui/src/tkTableCmd.h: Likewise.
* libgui/src/tkTableInitScript.h: Likewise.
Upgrade submodule, update code accordingly.
The gdb procedure varobj_delete() has dropped the "returned variable list"
feature, thus we can no longer use it to remove our tcl alias variables.
A new procedure variable_delete_tcl() is thus implemented to perform this
task before removing the gdb variable.
* gdbtk/generic/gdbtk_varobj.c: new procedure variable_delete_tcl().
(variable_delete): remove tcl alias variable using variable_delete_tcl().
Set running state in text window even if tracing features are enabled.
Failure to do so can segfault at rerun because the balloon variable may still
contain a dangling pointer to an item from the previous run.
Allow run commands to complete before target stops.
Latest gdb versions run command handlers return before the target stops,
even in synchronous mode. Insight must therefore monitor gdb events until
the target stops running.
* gdbtk/generic/gdbtk-hooks.c: include event-loop.h.
(gdbtk_call_command): Monitor gdb events after run command execution until
the inferior ptid has stopped or disappeared.
Patrick Monnerat [Wed, 26 Aug 2015 11:10:37 +0000 (13:10 +0200)]
plugins: unbundle tcl and adjust to latest cygwin.
* gdbtk/plugins/Make-rules: Use Tcl private include C flags from autotools
for Windows.
* gdbtk/plugins/rhabout/rhabout.c: Use __declspec(dllexport) for all
Windows builds.
(DllMain): Define for all native Windows builds. Remember Cygwin is not
a native Windows environment anymore.
Patrick Monnerat [Mon, 24 Aug 2015 16:44:02 +0000 (18:44 +0200)]
Action dialog: accept while-stepping without data collect.
Upgrade submodule.
* gdbtk/library/tracedlg.tcl (TraceDlg::get_actions): Preset empty data in
case scan does not set it.
(TraceDlg::edit): Preset empty data in case scan does not set it.
Set Editing option when creating an action dialog.
(gdb_add_tracepoint): Accept empty action for while-stepping.
(gdb_edit_tracepoint): Likewise.
* gdbtk/library/actiondlg.tcl: New option Editing.
(ActionDlg::ok): Check validity of step count for while-stepping.
Accept no data collection in while-stepping.
Rely on Editing option rather than data presence to determine add or modify
command to use.
Patrick Monnerat [Sat, 22 Aug 2015 11:58:01 +0000 (13:58 +0200)]
Action dialog: refine "change other".
* gdbtk/library/actiondlg.tcl (ActionDlg::change_other): Trim entry first.
Use regular expressions to match special tag names with multiple spaces.
Check if we are adding or deleting before any semantic check: this allows to
locally link to lists rather than copying them in and out, and simplifies
later processing.
Trim $ on the left only to check for a register name.
Patrick Monnerat [Sat, 22 Aug 2015 10:08:08 +0000 (12:08 +0200)]
Action dialog: fix "Collect Stack" processing.
* gdbtk/library/actiondlg.tcl (ActionDlg::change): Use lassign where possible.
Link to Collect and Variables rather than copying them in an out locally.
Use modified prototypes for procedures all_locals, all_regs and all_args.
Process the "Collect Stack" action specifically.
(ActionDlg::all_args): Arguments are listnamedel listnameadd.
Work directly on named arguments rather than copying in and returning a value.
(ActionDlg::all_locals): Likewise.
(ActionDlg::all_regs): Likewise.
(ActionDlg::change_other): Map expressions representing special tags to
the tag itself.
Compute add/del flag and copy in lists before checking the tag name.
Use new prototypes for procedures all_locals, all_regs and all_args.
Process "Collect Stack" properly.
Patrick Monnerat [Sat, 22 Aug 2015 09:27:20 +0000 (11:27 +0200)]
Consider C function arguments as such, not as local variables.
C function arguments do not have an argument class, but are regular local
variables flagged as being an argument. Check this flag in addition to
the symbol class.
* gdbtk/generic/gdbtk-stack.c (gdb_get_vars_command): Symbols with a
LOC_LOCAL, LOC_STATIC, LOC_REGISTER or LOC_COMPUTED class are arguments
if they are flagged as such.
Patrick Monnerat [Thu, 20 Aug 2015 15:35:58 +0000 (17:35 +0200)]
Actions dialog: do not propose expressions that cannot be collected.
This filters out uncollectable pseudo-registers and builds the stack collection
expression at run time.
* gdbtk/generic/gdbtk-register.c (get_register_collectable): New procedure:
lists whether a register is collectable or not.
(gdb_regspecial): New procedure: lists the register number of special
registers.
(gdb_register_info): New options "collectable" and "special".
* gdbtk/library/actiondlg.tcl (class ActionDlg): Do not preset StackCollect.
Define protected field StackPointer.
(ActionDlg::constructor): Filter out non-collectable registers.
Compute stack collection expression from current architecture's stack pointer.
Do not propose "All registers" if none exists.
Do not propose "Collect Stack" if the stack pointer is not collectable.
(ActionDlg::change_other): Fix a cut+pasted comment.
Patrick Monnerat [Wed, 19 Aug 2015 16:15:36 +0000 (18:15 +0200)]
Emulate trace start/stop hook.
This deprecated hook still exisits in gdb, but is not called anymore.
The optimal solution would be to have it replaced by an observer.
In the meantime, we check for a trace status change at each run/trace command
execution and eventually call the hook procedure explicitely.
* gdbtk/generic/gdbtk-hooks.c (gdbtk_add_hooks): Do not set the
deprecated_trace_start_stop_hook anymore.
(gdbtk_call_command): When a run/trace class command is executed, save the
trace running status before command and, if changed after command terminates,
call the hook procedure accordingly.
Patrick Monnerat [Wed, 19 Aug 2015 13:35:21 +0000 (15:35 +0200)]
Implement disabled tracepoint support.
* gdbtk/library/srctextwin.ith (class SrcTextWin): Add new type disabled_tp.
Fix a typo in comment.
* gdbtk/library/srctextwin.itb (SrcTextWin::constructor): Initialize
break_image for disabled_tp.
(SrcTextWin::build_popups): Initialize popups for disabled_tp.
Add Enable/Disable Tracepoint menu entries.
(SrcTextWin::enable_disable_src_tags): Bind <Enter>/<Motion>/<Leave> events of
disabled_tp_tag.
(SrcTextWin::config_win): Configure disabled_tp_tag.
Bind <Button-1> and <Button-3> event of disable_bp_tag.
(SrcTextWin::removeBreakTag): Match also disabled_tp tags.
(SrcTextWin::hasTP): Likewise.
(SrcTextWin::showTPBalloon): Determine tracepoint type.
Patrick Monnerat [Tue, 18 Aug 2015 17:43:49 +0000 (19:43 +0200)]
Implement tracepoint balloons.
* gdbtk/library/srctextwin.ith (SrcTextWin::showTPBalloon): New method.
* gdbtk/library/srctextwin.itb (SrcTextWin::showTPBalloon): New method.
(SrcTextWin::enable_disable_src_tags): Preset motion for tracepoints.
(SrcTextWin::motion): Check tracepoint type in addition to breakpoint and
variables.
Patrick Monnerat [Tue, 18 Aug 2015 14:28:44 +0000 (16:28 +0200)]
Enable tracepoint creation events.
* gdbtk/generic/gdbtk-bp.c (BREAKPOINT_IS_INTERESTING): Tracepoints are
interesting! This enables observer routing to Tcl events for tracepoints.
* gdbtk/library/bpwin.itb (BpWin::breakpoint): Ignore event if this is a
tracepoint window.
(BpWin::tracepoint): Ignore event if this is a breakpoint window.
Patrick Monnerat [Tue, 18 Aug 2015 12:23:36 +0000 (14:23 +0200)]
TraceDlg: replace WhileStepping flag by a presence test method.
This effectively allows deleting the while-stepping action and recreate a
new one.
* gdbtk/library/tracedlg.tcl: Delete obsolete WhileStepping flag.
(has_while_stepping): New method.
(add_action): Test duplicate while-stepping using method instead of flag.
(add_action_to_list): Do not set obsolete WhileStepping flag.
Patrick Monnerat [Tue, 18 Aug 2015 12:00:44 +0000 (14:00 +0200)]
Trace dialog: be more selective on action regular expressions.
Only allow the command word to appear at the beginning of the command: this
will avoid positive patches on variable names, such as understanding
"collect endOfFile" as and "end" command.
Patrick Monnerat [Mon, 17 Aug 2015 23:26:05 +0000 (00:26 +0100)]
Get tracepoint action subcommands too.
The while-stepping action may have collect subcommands: include them in
tracepoint info.
* gdbtk/generic/gdbtk-bp.c (gdb_get_action_list): New recursive procedure.
(gdb_get_tracepoint_info): Use gdb_get_action_list to recursively list
commands.
* gdbtk/library/tracedlg.tcl (add_all_actions): Process multiple while-stepping
subcommands.
(edit): Pass option -Data value as a list (to accept multiple expressions).
Patrick Monnerat [Sun, 16 Aug 2015 16:47:06 +0000 (17:47 +0100)]
Implement a subclass for the tracepoint window.
This prevents the confusion between the breakpont and the tracepoint windows.
It allows to have both open at the same time.
* gdbtk/library/managedwin.itb (ManagedWin::_open): Do not match an object
with a class if it's not the top class.
* gdbtk/library/srcbar.itcl (create_view_menu): Direct the tracepoints menu
entry to open a Tpwin tracepoint window.
Remove redundant "-tracepoints 0" on the breakpoints menu entry.
(create_window_buttons): The Tracepoints button opens a TpWin, not a Bpwin.
* gdbtk/library/srctextwin.itb (SrcTextWin::do_key): Key "tracepoints" opens
a TpWin, not a BpWin.
* gdbtk/library/tpwin.itcl: New file. Defines the TpWin subclass of BpWin.
* gdbtk/library/tclIndex: Defines the TpWin related indexes.
Patrick Monnerat [Sun, 16 Aug 2015 13:08:20 +0000 (14:08 +0100)]
Avoid accessing non-existing event "disposition" on a tracepoint.
* gdbtk/library/bpwin.itb (BpWin::bp_add): Get disposition event only for
breakpoints. Fix a typo.
(BpWin::bp_modify): Get disposition event only for breakpoints.
Patrick Monnerat [Fri, 14 Aug 2015 18:30:25 +0000 (20:30 +0200)]
Make the "Set Tracepoint" dialog working.
This is achieved by accepting configure arguments in the Labelledframe
constructor and avoiding a null-pointer dereferencing when no action is
associated.
* gdbtk/generic/gdbtk-bp.c (gdb_actions_command): When no action is given,
parse commands from a zero-length string.
* libgui/library/lframe.tcl (constructor): Accept arguments and configure.
Patrick Monnerat [Fri, 14 Aug 2015 15:52:28 +0000 (17:52 +0200)]
Make trace find feature working again.
The trace frame find GUI feature was connected to the deprecated_trace_find_hook
that gdb still defines, but does not call anymore.
The same function is now achieved via a traceframe_changed observer.
* gdbtk/generic/gdbtk-hooks.c:
(gdbtk_trace_find): Change prototype to match the traceframe_changed observer.
Pass the new arguments to tcl/tk.
(gdbtk_add_hooks): Connect gdbtk_trace_find as an observer, not a hook.
* gdbtk/library/interface.tcl (gdbtk_tcl_trace_find_hook):
Use observer parameters.
* gdbtk/library/srcbar.itcl (handle_trace_find_hook): Use observer parameters.
* gdbtk/library/srctextwin.it[bh] (trace_find_hook): Use observer parameters.
Patrick Monnerat [Thu, 13 Aug 2015 13:04:59 +0000 (15:04 +0200)]
gdbtk: adapt to the new explicit locations API
* gdb/generic/gdbtk-bp.c: Include location.h.
(gdb_get_breakpoint_info): Use event_location_to_string instead of
`addr_string'.
(gdb_set_bp): Use string_to_event_location and pass this to
create_breakpoint instead of an address string.
(tracepoint_exists): Use string_to_event_location and pass this to
decode_line_1 instead of an address string.
* gdb/generic/gdbtk-cmds.c: Include location.h.
(gdb_get_line_command): Use string_to_event_location and pass this to
decode_line_1 instead of an address string.
(gdb_get_file_command): Likewise.
(gdb_get_function_command): Likewise.
* gdb/generic/gdbtk-stack.c: Include location.h.
(gdb_get_vars_command):Use string_to_event_location and pass this to
decode_line_1 instead of an address string.
git://sourceware.org / insight.git / log