Remove struct buffer
authorTom Tromey <tom@tromey.com>
Fri, 16 Dec 2022 19:50:29 +0000 (12:50 -0700)
committerTom Tromey <tom@tromey.com>
Fri, 24 Feb 2023 18:53:03 +0000 (11:53 -0700)
I've long wanted to remove 'struct buffer', and thanks to Simon's
earlier patch, I was finally able to do so.  My feeling has been that
gdb already has several decent structures available for growing
strings: std::string of course, but also obstack and even objalloc
from BFD and dyn-string from libiberty.  The previous patches in this
series removed all the uses of struct buffer, so this one can remove
the code and the remaining #includes.

gdb/linux-nat.c
gdb/nat/linux-osdata.c
gdb/nat/linux-ptrace.c
gdb/nat/linux-ptrace.h
gdb/top.h
gdbserver/server.h
gdbsupport/Makefile.am
gdbsupport/Makefile.in
gdbsupport/buffer.cc [deleted file]
gdbsupport/buffer.h [deleted file]

index d6e69e61ef5791629d7155686158217c008cd114..e5391b9ce352062af68e3e3e9c8062b6574bc8eb 100644 (file)
@@ -60,7 +60,6 @@
 #include "symfile.h"
 #include "gdbsupport/agent.h"
 #include "tracepoint.h"
-#include "gdbsupport/buffer.h"
 #include "target-descriptions.h"
 #include "gdbsupport/filestuff.h"
 #include "objfiles.h"
index 2a160177ce320eb8d0dd5977d725f665bb20c089..3f63ade17f7fb3b1a559865568b122999acf6e25 100644 (file)
@@ -33,7 +33,6 @@
 #include <arpa/inet.h>
 
 #include "gdbsupport/xml-utils.h"
-#include "gdbsupport/buffer.h"
 #include <dirent.h>
 #include <sys/stat.h>
 #include "gdbsupport/filestuff.h"
index f312779aa2661b6c541772c6c97c83d5f1bf421d..1ddca72d5b16fe6162ba7f103ceb10d64aa37407 100644 (file)
@@ -20,7 +20,6 @@
 #include "linux-ptrace.h"
 #include "linux-procfs.h"
 #include "linux-waitpid.h"
-#include "gdbsupport/buffer.h"
 #ifdef HAVE_SYS_PROCFS_H
 #include <sys/procfs.h>
 #endif
index 19f473626de6faace0bdde2662f7b783c413cfdb..b838533cf2f6082f1233a11dca238b28e100abb6 100644 (file)
@@ -18,8 +18,6 @@
 #ifndef NAT_LINUX_PTRACE_H
 #define NAT_LINUX_PTRACE_H
 
-struct buffer;
-
 #include "nat/gdb_ptrace.h"
 #include "gdbsupport/gdb_wait.h"
 
index 9f9a0281be4ee9a18109ea727770d9d9337b7ebf..4972d5e256776478ec1686549a62f52983846e02 100644 (file)
--- a/gdb/top.h
+++ b/gdb/top.h
@@ -20,7 +20,6 @@
 #ifndef TOP_H
 #define TOP_H
 
-#include "gdbsupport/buffer.h"
 #include "gdbsupport/event-loop.h"
 #include "gdbsupport/next-iterator.h"
 #include "value.h"
index 7997d1a32e6e42de8b167bc91d4d06b0167ee659..730ec146530b18be5359333b41b069adaee7c0fb 100644 (file)
@@ -52,7 +52,6 @@ int vsnprintf(char *str, size_t size, const char *format, va_list ap);
 #  define PROG "gdbserver"
 #endif
 
-#include "gdbsupport/buffer.h"
 #include "gdbsupport/xml-utils.h"
 #include "regcache.h"
 #include "gdbsupport/gdb_signals.h"
index 83eb6c35846c0012522b12e48f1fcd7f9188b1fe..00524e9a5667cbedd59a963e5cbd9f6bfe645f44 100644 (file)
@@ -42,7 +42,6 @@ endif
 libgdbsupport_a_SOURCES = \
     agent.cc \
     btrace-common.cc \
-    buffer.cc \
     cleanups.cc \
     common-debug.cc \
     common-exceptions.cc \
index 6db76447d81b78906cbed50fd907134eb59f5120..89ed11062d537116a4e4059da018ca10be1c12a5 100644 (file)
@@ -147,7 +147,7 @@ libgdbsupport_a_LIBADD =
 @HAVE_PIPE_OR_PIPE2_TRUE@am__objects_1 = event-pipe.$(OBJEXT)
 @SELFTEST_TRUE@am__objects_2 = selftest.$(OBJEXT)
 am_libgdbsupport_a_OBJECTS = agent.$(OBJEXT) btrace-common.$(OBJEXT) \
-       buffer.$(OBJEXT) cleanups.$(OBJEXT) common-debug.$(OBJEXT) \
+       cleanups.$(OBJEXT) common-debug.$(OBJEXT) \
        common-exceptions.$(OBJEXT) common-inferior.$(OBJEXT) \
        common-regcache.$(OBJEXT) common-utils.$(OBJEXT) \
        environ.$(OBJEXT) errors.$(OBJEXT) event-loop.$(OBJEXT) \
@@ -369,7 +369,6 @@ noinst_LIBRARIES = libgdbsupport.a
 libgdbsupport_a_SOURCES = \
     agent.cc \
     btrace-common.cc \
-    buffer.cc \
     cleanups.cc \
     common-debug.cc \
     common-exceptions.cc \
@@ -478,7 +477,6 @@ distclean-compile:
 
 @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/agent.Po@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/btrace-common.Po@am__quote@
-@AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/buffer.Po@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/cleanups.Po@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/common-debug.Po@am__quote@
 @AMDEP_TRUE@@am__include@ @am__quote@./$(DEPDIR)/common-exceptions.Po@am__quote@
diff --git a/gdbsupport/buffer.cc b/gdbsupport/buffer.cc
deleted file mode 100644 (file)
index da6f3af..0000000
+++ /dev/null
@@ -1,178 +0,0 @@
-/* A simple growing buffer for GDB.
-  
-   Copyright (C) 2009-2023 Free Software Foundation, Inc.
-
-   This file is part of GDB.
-
-   This program is free software; you can redistribute it and/or modify
-   it under the terms of the GNU General Public License as published by
-   the Free Software Foundation; either version 3 of the License, or
-   (at your option) any later version.
-
-   This program is distributed in the hope that it will be useful,
-   but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-   GNU General Public License for more details.
-
-   You should have received a copy of the GNU General Public License
-   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
-
-#include "common-defs.h"
-#include "xml-utils.h"
-#include "buffer.h"
-#include "inttypes.h"
-void
-buffer_grow (struct buffer *buffer, const char *data, size_t size)
-{
-  char *new_buffer;
-  size_t new_buffer_size;
-
-  if (size == 0)
-    return;
-
-  new_buffer_size = buffer->buffer_size;
-
-  if (new_buffer_size == 0)
-    new_buffer_size = 1;
-
-  while (buffer->used_size + size > new_buffer_size)
-    new_buffer_size *= 2;
-  new_buffer = (char *) xrealloc (buffer->buffer, new_buffer_size);
-  memcpy (new_buffer + buffer->used_size, data, size);
-  buffer->buffer = new_buffer;
-  buffer->buffer_size = new_buffer_size;
-  buffer->used_size += size;
-}
-
-void
-buffer_free (struct buffer *buffer)
-{
-  if (!buffer)
-    return;
-
-  xfree (buffer->buffer);
-  buffer->buffer = NULL;
-  buffer->buffer_size = 0;
-  buffer->used_size = 0;
-}
-
-void
-buffer_init (struct buffer *buffer)
-{
-  memset (buffer, 0, sizeof (*buffer));
-}
-
-char*
-buffer_finish (struct buffer *buffer)
-{
-  char *ret = buffer->buffer;
-  buffer->buffer = NULL;
-  buffer->buffer_size = 0;
-  buffer->used_size = 0;
-  return ret;
-}
-
-void
-buffer_xml_printf (struct buffer *buffer, const char *format, ...)
-{
-  va_list ap;
-  const char *f;
-  const char *prev;
-  int percent = 0;
-
-  va_start (ap, format);
-
-  prev = format;
-  for (f = format; *f; f++)
-    {
-      if (percent)
-       {
-         char buf[32];
-         char *str = buf;
-         const char *f_old = f;
-         
-         switch (*f)
-           {
-           case 's':
-             str = va_arg (ap, char *);
-             break;
-           case 'd':
-             sprintf (str, "%d", va_arg (ap, int));
-             break;
-           case 'u':
-             sprintf (str, "%u", va_arg (ap, unsigned int));
-             break;
-           case 'x':
-             sprintf (str, "%x", va_arg (ap, unsigned int));
-             break;
-           case 'o':
-             sprintf (str, "%o", va_arg (ap, unsigned int));
-             break;
-           case 'l':
-             f++;
-             switch (*f)
-               {
-               case 'd':
-                 sprintf (str, "%ld", va_arg (ap, long));
-                 break;
-               case 'u':
-                 sprintf (str, "%lu", va_arg (ap, unsigned long));
-                 break;
-               case 'x':
-                 sprintf (str, "%lx", va_arg (ap, unsigned long));
-                 break;
-               case 'o':
-                 sprintf (str, "%lo", va_arg (ap, unsigned long));
-                 break;
-               case 'l':
-                 f++;
-                 switch (*f)
-                   {
-                   case 'd':
-                     sprintf (str, "%" PRId64,
-                              (int64_t) va_arg (ap, long long));
-                     break;
-                   case 'u':
-                     sprintf (str, "%" PRIu64,
-                              (uint64_t) va_arg (ap, unsigned long long));
-                     break;
-                   case 'x':
-                     sprintf (str, "%" PRIx64,
-                              (uint64_t) va_arg (ap, unsigned long long));
-                     break;
-                   case 'o':
-                     sprintf (str, "%" PRIo64,
-                              (uint64_t) va_arg (ap, unsigned long long));
-                     break;
-                   default:
-                     str = 0;
-                     break;
-                   }
-                 break;
-               default:
-                 str = 0;
-                 break;
-               }
-             break;
-           default:
-             str = 0;
-             break;
-           }
-
-         if (str)
-           {
-             buffer_grow (buffer, prev, f_old - prev - 1);
-             std::string p = xml_escape_text (str);
-             buffer_grow_str (buffer, p.c_str ());
-             prev = f + 1;
-           }
-         percent = 0;
-       }
-      else if (*f == '%')
-       percent = 1;
-    }
-
-  buffer_grow_str (buffer, prev);
-  va_end (ap);
-}
-
diff --git a/gdbsupport/buffer.h b/gdbsupport/buffer.h
deleted file mode 100644 (file)
index 7323c3a..0000000
+++ /dev/null
@@ -1,68 +0,0 @@
-/* A simple growing buffer for GDB.
-  
-   Copyright (C) 2009-2023 Free Software Foundation, Inc.
-
-   This file is part of GDB.
-
-   This program is free software; you can redistribute it and/or modify
-   it under the terms of the GNU General Public License as published by
-   the Free Software Foundation; either version 3 of the License, or
-   (at your option) any later version.
-
-   This program is distributed in the hope that it will be useful,
-   but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-   GNU General Public License for more details.
-
-   You should have received a copy of the GNU General Public License
-   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
-
-#ifndef COMMON_BUFFER_H
-#define COMMON_BUFFER_H
-
-struct buffer
-{
-  char *buffer;
-  size_t buffer_size; /* allocated size */
-  size_t used_size; /* actually used size */
-};
-
-/* Append DATA of size SIZE to the end of BUFFER.  Grows the buffer to
-   accommodate the new data.  */
-void buffer_grow (struct buffer *buffer, const char *data, size_t size);
-
-/* Append C to the end of BUFFER.  Grows the buffer to accommodate the
-   new data.  */
-
-static inline void
-buffer_grow_char (struct buffer *buffer, char c)
-{
-  buffer_grow (buffer, &c, 1);
-}
-
-/* Release any memory held by BUFFER.  */
-void buffer_free (struct buffer *buffer);
-
-/* Initialize BUFFER.  BUFFER holds no memory afterwards.  */
-void buffer_init (struct buffer *buffer);
-
-/* Return a pointer into BUFFER data, effectively transferring
-   ownership of the buffer memory to the caller.  Calling buffer_free
-   afterwards has no effect on the returned data.  */
-char* buffer_finish (struct buffer *buffer);
-
-/* Simple printf to buffer function.  Current implemented formatters:
-   %s - grow an xml escaped text in BUFFER.
-   %d - grow an signed integer in BUFFER.
-   %u - grow an unsigned integer in BUFFER.
-   %x - grow an unsigned integer formatted in hexadecimal in BUFFER.
-   %o - grow an unsigned integer formatted in octal in BUFFER.  */
-void buffer_xml_printf (struct buffer *buffer, const char *format, ...)
-  ATTRIBUTE_PRINTF (2, 3);
-
-#define buffer_grow_str(BUFFER,STRING)         \
-  buffer_grow (BUFFER, STRING, strlen (STRING))
-#define buffer_grow_str0(BUFFER,STRING)                        \
-  buffer_grow (BUFFER, STRING, strlen (STRING) + 1)
-
-#endif /* COMMON_BUFFER_H */