]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
Fix declaration of sparc_xfer_wcookie
authorSimon Marchi <simon.marchi@efficios.com>
Tue, 26 Nov 2019 17:12:04 +0000 (12:12 -0500)
committerSimon Marchi <simon.marchi@efficios.com>
Tue, 26 Nov 2019 19:29:21 +0000 (14:29 -0500)
When building sparc-nat.c with -Wmissing-declarations, we get:

      CXX    sparc-nat.o
    /home/smarchi/src/binutils-gdb/gdb/sparc-nat.c: In function ‘target_xfer_status sparc_xfer_wcookie(target_ops*, target_object, const char*, gdb_byte*, const gdb_byte*, ULONGEST, ULONGEST, ULONGEST*)’:
    /home/smarchi/src/binutils-gdb/gdb/sparc-nat.c:255:1: error: no previous declaration for ‘target_xfer_status sparc_xfer_wcookie(target_ops*, target_object, const char*, gdb_byte*, const gdb_byte*, ULONGEST, ULONGEST, ULONGEST*)’ [-Werror=missing-declarations]
     sparc_xfer_wcookie (struct target_ops *ops, enum target_object object,
     ^~~~~~~~~~~~~~~~~~

Indeed, the declaration is not in sync with the definition, fix that.

sparc_xfer_wcookie is used in sparc_target::xfer_partial.  sparc_target
is only used in the BSD sparc native files.  The error above was
obtained by running "make sparc-nat.o" on Linux with a cross-compiler
for sparc64-linux-gnu.  But I presume that if we were to build for real
with a BSD/sparc compiler, we would end up with an undefined symbol.

gdb/ChangeLog:

* sparc-nat.c (sparc_xfer_wcookie): Sync declaration with
definition.

Change-Id: Id41e706e5516968ff6a49469ddc48eceb29dd3ea

gdb/ChangeLog
gdb/sparc-nat.c

index ad0f981975ceb872a5d63f1d6279661b562ab87d..79c09f95e7f9353dc48640deeb17e0ff1d37c3a6 100644 (file)
@@ -1,3 +1,8 @@
+2019-11-26  Simon Marchi  <simon.marchi@efficios.com>
+
+       * sparc-nat.c (sparc_xfer_wcookie): Sync declaration with
+       definition.
+
 2019-11-26  Simon Marchi  <simon.marchi@efficios.com>
 
        * remote-sim.c (simulator_command): Make static, remove
index 0905c57438aaf8254c03ad029d1e6971253a5f39..9b7691337713286d5fc1cb8c7cefc93100d53ac6 100644 (file)
@@ -252,7 +252,7 @@ sparc_store_inferior_registers (struct regcache *regcache, int regnum)
    TARGET_OBJECT_WCOOKIE.  Fetch StackGhost Per-Process XOR cookie.  */
 
 enum target_xfer_status
-sparc_xfer_wcookie (struct target_ops *ops, enum target_object object,
+sparc_xfer_wcookie (enum target_object object,
                    const char *annex, gdb_byte *readbuf,
                    const gdb_byte *writebuf, ULONGEST offset, ULONGEST len,
                    ULONGEST *xfered_len)