This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: beginnings of Guile support
- From: Doug Evans <dje at sceeck dot org>
- To: Tom Tromey <tromey at redhat dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Mon, 9 Sep 2013 22:21:03 -0700
- Subject: Re: beginnings of Guile support
- Authentication-results: sourceware.org; auth=none
- References: <CAEei7h90tuXtBzVbNjtLQkv6ECA+EsdkYmt==MEMJcUf3m-bcA at mail dot gmail dot com> <871u4xhfca dot fsf at fleche dot redhat dot com>
On Mon, Sep 9, 2013 at 2:09 PM, Tom Tromey <tromey@redhat.com> wrote:
>>>>>> "Doug" == Doug Evans <dje@sceeck.org> writes:
>
> Doug> As a personal project, I'm adding scheme scripting to gdb.
>
> I'd rather not put this in. I think multiple scripting languages
> fragments the user community and leads to hard-to-debug problems.
>
> It also requires difficult cross-language integration. Your patch skips
> the hard bits, but I think that is cheating -- those have to be fixed
> first.
I did all I could do to say the patch was preliminary and that there was
still a lot to do.
Let's not get ahead of ourselves ...
If you have specific requirements, it would be good to know sooner
what they are.
> Doug> +slang_sourcer_func *
> Doug> +get_slang_sourcer (const char *file)
> Doug> +{
> Doug> + if (has_extension (file, ".py"))
> Doug> + {
> Doug> +#ifdef HAVE_PYTHON
> Doug> + return python_scripting_interface.source_script;
> Doug> +#else
> Doug> + return source_python_unsupported;
> Doug> +#endif
>
> I think there are too many #ifs of this form.
> It means the abstraction is incomplete.
The current incarnation isn't my favorite either.
Still working on that bit.
> Doug> /* Variables used to pass information between the Breakpoint
> Doug> constructor and the breakpoint-created hook function. */
> Doug> -breakpoint_object *bppy_pending_object;
> Doug> +gdbpy_breakpoint_object *bppy_pending_object;
>
> Renamings ought to be separate patches.
When the patch is ready to be submitted RFA, it will be split up.
I was hoping to leave such things as givens.
> Doug> -/* Helper function that overrides this Python object's
> Doug> - PyObject_GenericSetAttr to allow extra validation of the attribute
> Doug> - being set. */
> Doug> -
> Doug> -static int
> Doug> -local_setattro (PyObject *self, PyObject *name, PyObject *v)
> Doug> -{
> Doug> - breakpoint_object *obj = (breakpoint_object *) self;
> [...]
>
> I'm not sure why this was deleted, but it also seems like it ought to be
> a separate patch, or maybe dropped.
Righto.
> Doug> +/* INCOMPLETE TODO LIST:
>
> I skimmed the guile code, but didn't read it heavily.
>
> I think anything that calls into Guile from gdb ought to prevent
> continuation capture. Otherwise madness will ensue. I don't know how
> you do that in Guile but I didn't see anything obvious in the code.
All calls to Scheme are gated through specific routines.
Still working on all the details.