Remove useless NULL check in python.c

I noticed that do_start_initialization, in python.c, checks the result
of xmalloc.  However, xmalloc cannot fail, so this check is useless.
This patch also changes the code to use XNEWVEC.

gdb/ChangeLog
2020-02-19  Tom Tromey  <tromey@adacore.com>

	* python/python.c (do_start_initialization): Use XNEWVEC.  Remove
	NULL check.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 0077a9824aa4410e11079e0a052a914ba2215558..e5df6bf48c96e45eaea466ba6df195d3b7cb0edf 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2020-02-19  Tom Tromey  <tromey@adacore.com>
+
+       * python/python.c (do_start_initialization): Use XNEWVEC.  Remove
+       NULL check.
+
 2020-02-19  Maciej W. Rozycki  <macro@wdc.com>
 
        * NEWS: Mention RISC-V GNU/Linux GDBserver support.

diff --git a/gdb/python/python.c b/gdb/python/python.c
index 27d6042c618887cf1a31546d7737bf805196824e..fbbc159ede3ee5da46c4b8f85bae364423ed5b58 100644 (file)
--- a/gdb/python/python.c
+++ b/gdb/python/python.c
@@ -1679,12 +1679,7 @@ do_start_initialization ()
   std::string oldloc = setlocale (LC_ALL, NULL);
   setlocale (LC_ALL, "");
   progsize = strlen (progname.get ());
-  progname_copy = (wchar_t *) xmalloc ((progsize + 1) * sizeof (wchar_t));
-  if (!progname_copy)
-    {
-      fprintf (stderr, "out of memory\n");
-      return false;
-    }
+  progname_copy = XNEWVEC (wchar_t, progsize + 1);
   count = mbstowcs (progname_copy, progname.get (), progsize + 1);
   if (count == (size_t) -1)
     {