gdb: Fix ATTRIBUTE_NONNULL usage
authorPedro Alves <palves@redhat.com>
Wed, 8 Mar 2017 15:52:16 +0000 (15:52 +0000)
committerPedro Alves <palves@redhat.com>
Wed, 8 Mar 2017 15:52:16 +0000 (15:52 +0000)
Should fix the build failure with Clang mentioned at
<https://sourceware.org/bugzilla/show_bug.cgi?id=21206#c2>:

 In file included from ../../binutils-gdb/gdb/dwarf2read.c:72:
 ../../binutils-gdb/gdb/common/gdb_unlinker.h:35:35: error: '__nonnull__' attribute is invalid for the implicit this argument
   unlinker (const char *filename) ATTRIBUTE_NONNULL (1)
   ^                  ~
 ../../binutils-gdb/gdb/../include/ansidecl.h:169:48: note: expanded from macro 'ATTRIBUTE_NONNULL'
 #  define ATTRIBUTE_NONNULL(m) __attribute__ ((__nonnull__ (m)))

gdb/ChangeLog:
2017-03-08  Pedro Alves  <palves@redhat.com>

PR 21206
* common/gdb_unlinker.h (unlinker::unlinker): Attribute nonnull
goes to argument 2, not 1.

gdb/ChangeLog
gdb/common/gdb_unlinker.h

index 92847f721778501fac3ea4f5d43c3b9ea9877c56..ce37aa79f6c31e788bc184fff3be4563af75beca 100644 (file)
@@ -1,3 +1,9 @@
+2017-03-08  Pedro Alves  <palves@redhat.com>
+
+       PR 21206
+       * common/gdb_unlinker.h (unlinker::unlinker): Attribute nonnull
+       goes to argument 2, not 1.
+
 2017-03-08  Pedro Alves  <palves@redhat.com>
 
        PR cli/21218
index 0f3b98a54ff5a34f7d4b28c754a7362130354bba..e93fa79e167ecceed51ba6f372d4da718b601555 100644 (file)
@@ -32,7 +32,7 @@ class unlinker
 {
  public:
 
-  unlinker (const char *filename) ATTRIBUTE_NONNULL (1)
+  unlinker (const char *filename) ATTRIBUTE_NONNULL (2)
     : m_filename (filename)
   {
     gdb_assert (filename != NULL);