This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[RFA] (cli/cli-cmds.c) ARI fix: Avoid assignment inside if statement
- From: "Pierre Muller" <pierre dot muller at ics-cnrs dot unistra dot fr>
- To: <gdb-patches at sourceware dot org>
- Date: Sun, 23 Dec 2012 19:59:49 +0100
- Subject: [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