From: Mark Kettenis <mark.kettenis@xs4all.nl>
To: gdb-patches@sourceware.org
Subject: [RFA] Cleanup mi_cmd_data_write_register_values
Date: Mon, 01 Jan 2007 01:45:00 -0000 [thread overview]
Message-ID: <200701010145.l011j8FY027694@brahms.sibelius.xs4all.nl> (raw)
The next victim in my vendetta agains deprecated_xxx. Looks like this
function pretty was pretty much rolling its own
regcache_cooked_write_signed(), so why not use that?
ok?
Mark
Index: ChangeLog
from Mark Kettenis <kettenis@gnu.org>
* mi/mi-main.c (mi_cmd_data_write_register_values): Use
regcache_cooked_write_signed instead of
deprecated_write_register_bytes.
Index: mi/mi-main.c
===================================================================
RCS file: /cvs/src/src/gdb/mi/mi-main.c,v
retrieving revision 1.86
diff -u -p -r1.86 mi-main.c
--- mi/mi-main.c 17 Nov 2006 19:30:41 -0000 1.86
+++ mi/mi-main.c 1 Jan 2007 01:42:23 -0000
@@ -545,10 +545,7 @@ get_register (int regnum, int format)
enum mi_cmd_result
mi_cmd_data_write_register_values (char *command, char **argv, int argc)
{
- int regnum;
- int i;
- int numregs;
- LONGEST value;
+ int numregs, i;
char format;
/* Note that the test for a valid register must include checking the
@@ -587,26 +584,18 @@ mi_cmd_data_write_register_values (char
for (i = 1; i < argc; i = i + 2)
{
- regnum = atoi (argv[i]);
+ int regnum = atoi (argv[i]);
- if (regnum >= 0
- && regnum < numregs
- && REGISTER_NAME (regnum) != NULL
- && *REGISTER_NAME (regnum) != '\000')
+ if (regnum >= 0 && regnum < numregs
+ && REGISTER_NAME (regnum) && *REGISTER_NAME (regnum))
{
- void *buffer;
- struct cleanup *old_chain;
+ LONGEST value;
- /* Get the value as a number */
+ /* Get the value as a number. */
value = parse_and_eval_address (argv[i + 1]);
- /* Get the value into an array */
- buffer = xmalloc (DEPRECATED_REGISTER_SIZE);
- old_chain = make_cleanup (xfree, buffer);
- store_signed_integer (buffer, DEPRECATED_REGISTER_SIZE, value);
+
/* Write it down */
- deprecated_write_register_bytes (DEPRECATED_REGISTER_BYTE (regnum), buffer, register_size (current_gdbarch, regnum));
- /* Free the buffer. */
- do_cleanups (old_chain);
+ regcache_cooked_write_signed (current_regcache, regnum, value);
}
else
{
next reply other threads:[~2007-01-01 1:45 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-01-01 1:45 Mark Kettenis [this message]
2007-01-01 3:07 ` Daniel Jacobowitz
2007-01-01 23:33 ` Nick Roberts
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200701010145.l011j8FY027694@brahms.sibelius.xs4all.nl \
--to=mark.kettenis@xs4all.nl \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox