From 5c328c057e963faacc50796c650a5feb06762c67 Mon Sep 17 00:00:00 2001 From: Yao Qi Date: Sun, 23 Feb 2014 12:24:08 +0800 Subject: [PATCH] Remove TARGET_XFER_STATUS_ERROR_P This patch removes macro TARGET_XFER_STATUS_ERROR_P, as Pedro pointed out during patches review that TARGET_XFER_STATUS_ERROR_P tends to be unnecessary. gdb: 2014-02-24 Yao Qi * target.h (TARGET_XFER_STATUS_ERROR_P): Remove. * corefile.c (read_memory): Adjusted. * target.c (target_write_with_progress): Adjusted. --- gdb/ChangeLog | 6 ++++++ gdb/corefile.c | 9 +++------ gdb/target.c | 7 ++----- gdb/target.h | 2 -- 4 files changed, 11 insertions(+), 13 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 3b199ac9b63..01be0a30555 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2014-02-24 Yao Qi + + * target.h (TARGET_XFER_STATUS_ERROR_P): Remove. + * corefile.c (read_memory): Adjusted. + * target.c (target_write_with_progress): Adjusted. + 2014-02-23 Yao Qi Revert two patches: diff --git a/gdb/corefile.c b/gdb/corefile.c index 048669b9d2e..815adaf46a0 100644 --- a/gdb/corefile.c +++ b/gdb/corefile.c @@ -260,13 +260,10 @@ read_memory (CORE_ADDR memaddr, gdb_byte *myaddr, ssize_t len) memaddr + xfered, len - xfered, &xfered_len); - if (status == TARGET_XFER_EOF) - memory_error (TARGET_XFER_E_IO, memaddr + xfered); + if (status != TARGET_XFER_OK) + memory_error (status == TARGET_XFER_EOF ? TARGET_XFER_E_IO : status, + memaddr + xfered); - if (TARGET_XFER_STATUS_ERROR_P (status)) - memory_error (status, memaddr + xfered); - - gdb_assert (status == TARGET_XFER_OK); xfered += xfered_len; QUIT; } diff --git a/gdb/target.c b/gdb/target.c index 0f3bd301652..60a11dddb6c 100644 --- a/gdb/target.c +++ b/gdb/target.c @@ -2002,12 +2002,9 @@ target_write_with_progress (struct target_ops *ops, offset + xfered, len - xfered, &xfered_len); - if (status == TARGET_XFER_EOF) - return xfered; - if (TARGET_XFER_STATUS_ERROR_P (status)) - return -1; + if (status != TARGET_XFER_OK) + return status == TARGET_XFER_EOF ? xfered : -1; - gdb_assert (status == TARGET_XFER_OK); if (progress) (*progress) (xfered_len, baton); diff --git a/gdb/target.h b/gdb/target.h index 6cc13370759..4254609fca8 100644 --- a/gdb/target.h +++ b/gdb/target.h @@ -225,8 +225,6 @@ enum target_xfer_status /* Keep list in sync with target_xfer_error_to_string. */ }; -#define TARGET_XFER_STATUS_ERROR_P(STATUS) ((STATUS) < TARGET_XFER_EOF) - /* Return the string form of ERR. */ extern const char *target_xfer_status_to_string (enum target_xfer_status err); -- 2.30.2