I noticed a couple of unnecessary casts in cli-cmds.c.  This patch
removes them.
Tested by rebuilding.  I'm checking this in.
gdb/ChangeLog
2018-09-05  Tom Tromey  <tom@tromey.com>
	* cli/cli-cmds.c (shell_escape, edit_command): Remove cast.
+2018-09-05  Tom Tromey  <tom@tromey.com>
+
+       * cli/cli-cmds.c (shell_escape, edit_command): Remove cast.
+
 2018-09-05  Tom de Vries  <tdevries@suse.de>
 
        * dwarf2loc.c (sect_variable_value): Call indirect_synthetic_pointer
 
 
       close_most_fds ();
 
-      if ((user_shell = (char *) getenv ("SHELL")) == NULL)
+      if ((user_shell = getenv ("SHELL")) == NULL)
        user_shell = "/bin/sh";
 
       /* Get the name of the shell for arg0.  */
         error (_("No line number known for %s."), arg);
     }
 
-  if ((editor = (char *) getenv ("EDITOR")) == NULL)
-      editor = "/bin/ex";
+  if ((editor = getenv ("EDITOR")) == NULL)
+    editor = "/bin/ex";
 
   fn = symtab_to_fullname (sal.symtab);