* [Bug guile/17203] It should be an error to register a pre-existing parameter.
2014-07-27 4:38 [Bug guile/17203] New: It should be an error to register a pre-existing parameter xdje42 at gmail dot com
@ 2014-07-28 13:30 ` brobecker at gnat dot com
2014-07-28 14:30 ` xdje42 at gmail dot com
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: brobecker at gnat dot com @ 2014-07-28 13:30 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=17203
Joel Brobecker <brobecker at gnat dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |brobecker at gnat dot com
--- Comment #1 from Joel Brobecker <brobecker at gnat dot com> ---
Hi Doug,
How long do you think before we have a fix for this?
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug guile/17203] It should be an error to register a pre-existing parameter.
2014-07-27 4:38 [Bug guile/17203] New: It should be an error to register a pre-existing parameter xdje42 at gmail dot com
2014-07-28 13:30 ` [Bug guile/17203] " brobecker at gnat dot com
@ 2014-07-28 14:30 ` xdje42 at gmail dot com
2014-07-28 14:35 ` brobecker at adacore dot com
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: xdje42 at gmail dot com @ 2014-07-28 14:30 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=17203
--- Comment #2 from Doug Evans <xdje42 at gmail dot com> ---
(In reply to Joel Brobecker from comment #1)
> Hi Doug,
>
> How long do you think before we have a fix for this?
Patch is posted here:
https://sourceware.org/ml/gdb-patches/2014-07/msg00690.html
It can be checked in today IMO.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug guile/17203] It should be an error to register a pre-existing parameter.
2014-07-27 4:38 [Bug guile/17203] New: It should be an error to register a pre-existing parameter xdje42 at gmail dot com
2014-07-28 13:30 ` [Bug guile/17203] " brobecker at gnat dot com
2014-07-28 14:30 ` xdje42 at gmail dot com
@ 2014-07-28 14:35 ` brobecker at adacore dot com
2014-07-28 14:50 ` xdje42 at gmail dot com
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: brobecker at adacore dot com @ 2014-07-28 14:35 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=17203
--- Comment #3 from brobecker at adacore dot com ---
> Patch is posted here:
> https://sourceware.org/ml/gdb-patches/2014-07/msg00690.html
> It can be checked in today IMO.
Awesome! We don't need to rush this patch just because of the release,
but I think I will create the release as soon as this one is in.
Thanks, Doug.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug guile/17203] It should be an error to register a pre-existing parameter.
2014-07-27 4:38 [Bug guile/17203] New: It should be an error to register a pre-existing parameter xdje42 at gmail dot com
` (2 preceding siblings ...)
2014-07-28 14:35 ` brobecker at adacore dot com
@ 2014-07-28 14:50 ` xdje42 at gmail dot com
2014-07-29 2:28 ` cvs-commit at gcc dot gnu.org
2024-01-19 12:27 ` ssbssa at sourceware dot org
5 siblings, 0 replies; 7+ messages in thread
From: xdje42 at gmail dot com @ 2014-07-28 14:50 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=17203
--- Comment #4 from Doug Evans <xdje42 at gmail dot com> ---
On a related topic,
One thing I'd like to add is the ability to export native gdb parameters to
Scheme.
One reason is I'd like to write a function that executes a gdb command with a
parameter temporarily set to a specified value. In order to easily do that,
i.e., without having to have a lot of parameter-specific code, I want to be
able to read and write parameters from Scheme and not using gdb set/show
commands.
Implementing this will involve a bit of work for parameters that post-process
writes.
The point of noting this here is that in this case we're not registering a new
parameter (using the parlance of "register-parameter!"), just exporting to
Scheme an existing one.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug guile/17203] It should be an error to register a pre-existing parameter.
2014-07-27 4:38 [Bug guile/17203] New: It should be an error to register a pre-existing parameter xdje42 at gmail dot com
` (3 preceding siblings ...)
2014-07-28 14:50 ` xdje42 at gmail dot com
@ 2014-07-29 2:28 ` cvs-commit at gcc dot gnu.org
2024-01-19 12:27 ` ssbssa at sourceware dot org
5 siblings, 0 replies; 7+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2014-07-29 2:28 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=17203
--- Comment #5 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "gdb and binutils".
The branch, master has been updated
via 7ebdbe9292e4b696740b021938369adb1484da27 (commit)
from 385f635ae0be0077dad606c12118bbe0f886f28f (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=7ebdbe9292e4b696740b021938369adb1484da27
commit 7ebdbe9292e4b696740b021938369adb1484da27
Author: Doug Evans <xdje42@gmail.com>
Date: Mon Jul 28 19:20:30 2014 -0700
PR guile/17203
* guile/scm-param.c (pascm_parameter_defined_p): New function.
(gdbscm_register_parameter_x): Call it. Raise error for pre-existing
parameters.
testsuite/
* gdb.guile/scm-parameter.exp: Add tests for trying to create
previously existing parameter, and previously ambiguously spelled
parameter.
-----------------------------------------------------------------------
Summary of changes:
gdb/ChangeLog | 7 +++++++
gdb/guile/scm-param.c | 27 ++++++++++++++++++++++++++-
gdb/testsuite/ChangeLog | 7 +++++++
gdb/testsuite/gdb.guile/scm-parameter.exp | 28 ++++++++++++++++++++++++++++
4 files changed, 68 insertions(+), 1 deletions(-)
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug guile/17203] It should be an error to register a pre-existing parameter.
2014-07-27 4:38 [Bug guile/17203] New: It should be an error to register a pre-existing parameter xdje42 at gmail dot com
` (4 preceding siblings ...)
2014-07-29 2:28 ` cvs-commit at gcc dot gnu.org
@ 2024-01-19 12:27 ` ssbssa at sourceware dot org
5 siblings, 0 replies; 7+ messages in thread
From: ssbssa at sourceware dot org @ 2024-01-19 12:27 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=17203
Hannes Domani <ssbssa at sourceware dot org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |ssbssa at sourceware dot org
--- Comment #6 from Hannes Domani <ssbssa at sourceware dot org> ---
(In reply to Sourceware Commits from comment #5)
> This is an automated email from the git hooks/post-receive script. It was
> generated because a ref change was pushed to the repository containing
> the project "gdb and binutils".
>
> The branch, master has been updated
> via 7ebdbe9292e4b696740b021938369adb1484da27 (commit)
> from 385f635ae0be0077dad606c12118bbe0f886f28f (commit)
>
> Those revisions listed above that are new to this repository have
> not appeared on any other notification email; so we list those
> revisions in full, below.
>
> - Log -----------------------------------------------------------------
> https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;
> h=7ebdbe9292e4b696740b021938369adb1484da27
>
> commit 7ebdbe9292e4b696740b021938369adb1484da27
> Author: Doug Evans <xdje42@gmail.com>
> Date: Mon Jul 28 19:20:30 2014 -0700
>
> PR guile/17203
>
> * guile/scm-param.c (pascm_parameter_defined_p): New function.
> (gdbscm_register_parameter_x): Call it. Raise error for pre-existing
> parameters.
Can this be closed?
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread