public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug guile/17203] New: It should be an error to register a pre-existing parameter.
@ 2014-07-27  4:38 xdje42 at gmail dot com
  2014-07-28 13:30 ` [Bug guile/17203] " brobecker at gnat dot com
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: xdje42 at gmail dot com @ 2014-07-27  4:38 UTC (permalink / raw)
  To: gdb-prs

https://sourceware.org/bugzilla/show_bug.cgi?id=17203

            Bug ID: 17203
           Summary: It should be an error to register a pre-existing
                    parameter.
           Product: gdb
           Version: HEAD
            Status: NEW
          Severity: normal
          Priority: P2
         Component: guile
          Assignee: unassigned at sourceware dot org
          Reporter: xdje42 at gmail dot com

It should be an error to register a pre-existing parameter.

E.g., one can't extend a native gdb parameter.
And changing the meaning/operation of a pre-existing parameter will just
confuse users.

I'm not sure Python parameters can be fixed, but Guile parameters can be before
7.8 goes out.

-- 
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 ` 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

end of thread, other threads:[~2024-01-19 12:28 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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
2014-07-29  2:28 ` cvs-commit at gcc dot gnu.org
2024-01-19 12:27 ` ssbssa at sourceware dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).