This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[RFA] (cli/cli-cmds.c) ARI fix: Avoid assignment inside if statement


  Looks almost obvious, but
as it's the first of that kind,
I thought I will send it as RFA nonetheless...
  
  Can similar changes be committed as obvious?
They usually require some formatting changes,
but generated behavior should not change.


Pierre Muller
as ARI maintainer


2012-12-20  Pierre Muller  <muller@sourceware.org>

        ARI fixes: Assignment within if rule.
        * cli/cli-cmds.c (shell_escape): Do not set variable value inside
        if statement.
        (edit_command): Likewise.

Index: src/gdb/cli/cli-cmds.c
===================================================================
RCS file: /cvs/src/src/gdb/cli/cli-cmds.c,v
retrieving revision 1.143
diff -u -p -r1.143 cli-cmds.c
--- src/gdb/cli/cli-cmds.c      18 Dec 2012 19:27:35 -0000      1.143
+++ src/gdb/cli/cli-cmds.c      20 Dec 2012 16:10:27 -0000
@@ -724,11 +724,13 @@ shell_escape (char *arg, int from_tty)
 #else /* Can fork.  */
   int status, pid;

-  if ((pid = vfork ()) == 0)
+  pid = vfork ();
+  if (pid == 0)
     {
       const char *p, *user_shell;

-      if ((user_shell = (char *) getenv ("SHELL")) == NULL)
+      user_shell = (char *) getenv ("SHELL");
+      if (user_shell == NULL)
        user_shell = "/bin/sh";

       /* Get the name of the shell for arg0.  */
@@ -837,7 +839,8 @@ edit_command (char *arg, int from_tty)
         error (_("No line number known for %s."), arg);
     }

-  if ((editor = (char *) getenv ("EDITOR")) == NULL)
+  editor = (char *) getenv ("EDITOR");
+  if (editor == NULL)
       editor = "/bin/ex";

   /* If we don't already know the full absolute file name of the


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]