Add "const" to a few locals in gdb
authorTom Tromey <tom@tromey.com>
Mon, 24 Sep 2018 12:15:17 +0000 (06:15 -0600)
committerTom Tromey <tom@tromey.com>
Mon, 24 Sep 2018 12:17:13 +0000 (06:17 -0600)
I noticed that some code in gdb was doing:

    char *mumble = getenv (...)

However, using "const char *" here would be clearer.
This patch fixes the instances I could readily build.

Tested by rebuilding.

gdb/ChangeLog
2018-09-24  Tom Tromey  <tom@tromey.com>

* common/pathstuff.c (get_standard_cache_dir): Make
"xdg_cache_home" and "home" const.
* top.c (init_history): Make "tmpenv" const.
* main.c (get_init_files): Make "homedir" const.

gdb/ChangeLog
gdb/common/pathstuff.c
gdb/main.c
gdb/top.c

index e8005c0..13fac92 100644 (file)
@@ -1,3 +1,10 @@
+2018-09-24  Tom Tromey  <tom@tromey.com>
+
+       * common/pathstuff.c (get_standard_cache_dir): Make
+       "xdg_cache_home" and "home" const.
+       * top.c (init_history): Make "tmpenv" const.
+       * main.c (get_init_files): Make "homedir" const.
+
 2018-09-23  Tom Tromey  <tom@tromey.com>
 
        PR python/18852:
index 3dd58e3..82905c9 100644 (file)
@@ -171,7 +171,7 @@ get_standard_cache_dir ()
 #endif
 
 #ifndef __APPLE__
-  char *xdg_cache_home = getenv ("XDG_CACHE_HOME");
+  const char *xdg_cache_home = getenv ("XDG_CACHE_HOME");
   if (xdg_cache_home != NULL)
     {
       /* Make sure the path is absolute and tilde-expanded.  */
@@ -180,7 +180,7 @@ get_standard_cache_dir ()
     }
 #endif
 
-  char *home = getenv ("HOME");
+  const char *home = getenv ("HOME");
   if (home != NULL)
     {
       /* Make sure the path is absolute and tilde-expanded.  */
index 61644cd..1552e95 100644 (file)
@@ -208,7 +208,7 @@ get_init_files (const char **system_gdbinit,
   if (!initialized)
     {
       struct stat homebuf, cwdbuf, s;
-      char *homedir;
+      const char *homedir;
 
       if (SYSTEM_GDBINIT[0])
        {
index 0a4d36c..4a0fedb 100644 (file)
--- a/gdb/top.c
+++ b/gdb/top.c
@@ -1846,7 +1846,7 @@ set_verbose (const char *args, int from_tty, struct cmd_list_element *c)
 void
 init_history (void)
 {
-  char *tmpenv;
+  const char *tmpenv;
 
   tmpenv = getenv ("GDBHISTSIZE");
   if (tmpenv)