Prefer xpalloc to doubling buffers by hand

* src/lread.c (grow_read_buffer): New function, which uses xpalloc.
(read1): Use it for simplicity.
* src/macros.c (store_kbd_macro_char):
* src/minibuf.c (read_minibuf_noninteractive):
* src/term.c (encode_terminal_code):
* src/xrdb.c (magic_db):
Prefer xpalloc to growing buffers by hand.
This doesn’t fix any bugs, but simplifies the code a bit.

1 files changed, 2 insertions, 6 deletions

diff --git a/src/xrdb.c b/src/xrdb.c
index ce6e7d21edb..10bc76986e6 100644
--- a/src/xrdb.c
+++ b/src/xrdb.c
@@ -177,12 +177,8 @@ magic_db (const char *string, ptrdiff_t string_len, const char *class,
 
       /* Do we have room for this component followed by a '\0'?  */
       if (path_size - path_len <= next_len)
-        {
-          if (min (PTRDIFF_MAX, SIZE_MAX) / 2 - 1 - path_len < next_len)
-            memory_full (SIZE_MAX);
-          path_size = (path_len + next_len + 1) * 2;
-          path = xrealloc (path, path_size);
-        }
+        path = xpalloc (path, &path_size, path_len - path_size + next_len + 1,
+                        -1, sizeof *path);
 
       memcpy (path + path_len, next, next_len);
       path_len += next_len;