From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 12849 invoked by alias); 29 Jul 2008 19:47:32 -0000 Received: (qmail 12840 invoked by uid 22791); 29 Jul 2008 19:47:31 -0000 X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (66.187.233.31) by sourceware.org (qpsmtpd/0.31) with ESMTP; Tue, 29 Jul 2008 19:47:14 +0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id m6TJlA2J020444; Tue, 29 Jul 2008 15:47:10 -0400 Received: from pobox.corp.redhat.com (pobox.corp.redhat.com [10.11.255.20]) by int-mx1.corp.redhat.com (8.13.1/8.13.1) with ESMTP id m6TJl9qt019262; Tue, 29 Jul 2008 15:47:09 -0400 Received: from opsy.redhat.com (vpn-10-116.bos.redhat.com [10.16.10.116]) by pobox.corp.redhat.com (8.13.1/8.13.1) with ESMTP id m6TJl9st014226; Tue, 29 Jul 2008 15:47:09 -0400 Received: by opsy.redhat.com (Postfix, from userid 500) id ADD175086F2; Tue, 29 Jul 2008 13:47:08 -0600 (MDT) To: Michael Snyder Cc: gdb-patches@sourceware.org Subject: Re: RFA: clean up edit_command References: <1217299668.3549.576.camel@localhost.localdomain> From: Tom Tromey Reply-To: Tom Tromey X-Attribution: Tom Date: Tue, 29 Jul 2008 19:47:00 -0000 In-Reply-To: <1217299668.3549.576.camel@localhost.localdomain> (Michael Snyder's message of "Mon\, 28 Jul 2008 19\:47\:48 -0700") Message-ID: User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2008-07/txt/msg00545.txt.bz2 >>>>> "Michael" == Michael Snyder writes: Michael> OK, but don't you want to remove cmdlen as well? Michael> Doesn't seem to be used any more... Sure. Here's an updated patch. Built and regression tested on compile-farm (x86-64). Ok? Tom 2008-07-28 Tom Tromey * cli/cli-cmds.c (edit_command): Remove unused variables. Delete dead code. Fix indentation. diff --git a/gdb/cli/cli-cmds.c b/gdb/cli/cli-cmds.c index 8dc178e..d9d2c56 100644 --- a/gdb/cli/cli-cmds.c +++ b/gdb/cli/cli-cmds.c @@ -615,12 +615,10 @@ edit_command (char *arg, int from_tty) struct symtab_and_line sal; struct symbol *sym; char *arg1; - int cmdlen, log10; - unsigned m; char *editor; char *p, *fn; - /* Pull in the current default source line if necessary */ + /* Pull in the current default source line if necessary. */ if (arg == 0) { set_default_source_symtab_and_line (); @@ -638,17 +636,22 @@ edit_command (char *arg, int from_tty) else { - /* Now should only be one argument -- decode it in SAL */ + /* Now should only be one argument -- decode it in SAL. */ arg1 = arg; sals = decode_line_1 (&arg1, 0, 0, 0, 0, 0); - if (! sals.nelts) return; /* C++ */ - if (sals.nelts > 1) { - ambiguous_line_spec (&sals); - xfree (sals.sals); - return; - } + if (! sals.nelts) + { + /* C++ */ + return; + } + if (sals.nelts > 1) + { + ambiguous_line_spec (&sals); + xfree (sals.sals); + return; + } sal = sals.sals[0]; xfree (sals.sals); @@ -656,7 +659,7 @@ edit_command (char *arg, int from_tty) if (*arg1) error (_("Junk at end of line specification.")); - /* if line was specified by address, + /* If line was specified by address, first print exactly which line, and which file. In this case, sal.symtab == 0 means address is outside of all known source files, not that user failed to give a filename. */ @@ -686,10 +689,6 @@ edit_command (char *arg, int from_tty) if ((editor = (char *) getenv ("EDITOR")) == NULL) editor = "/bin/ex"; - /* Approximate base-10 log of line to 1 unit for digit count */ - for(log10=32, m=0x80000000; !(sal.line & m) && log10>0; log10--, m=m>>1); - log10 = 1 + (int)((log10 + (0 == ((m-1) & sal.line)))/3.32192809); - /* If we don't already know the full absolute file name of the source file, find it now. */ if (!sal.symtab->fullname) @@ -704,8 +703,8 @@ edit_command (char *arg, int from_tty) /* Quote the file name, in case it has whitespace or other special characters. */ p = xstrprintf ("%s +%d \"%s\"", editor, sal.line, fn); - shell_escape(p, from_tty); - xfree(p); + shell_escape (p, from_tty); + xfree (p); } static void