From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-11780-listarch-gdb-patches=sourceware.cygnus.com@sources.redhat.com>
Received: (qmail 11022 invoked by alias); 10 Jan 2002 15:15:34 -0000
Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:gdb-patches-subscribe@sources.redhat.com>
List-Archive: <http://sources.redhat.com/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sources.redhat.com>
List-Help: <mailto:gdb-patches-help@sources.redhat.com>, <http://sources.redhat.com/ml/#faqs>
Sender: gdb-patches-owner@sources.redhat.com
Received: (qmail 10913 invoked from network); 10 Jan 2002 15:15:29 -0000
Received: from unknown (HELO localhost.cygnus.com) (24.114.42.213)
  by sources.redhat.com with SMTP; 10 Jan 2002 15:15:29 -0000
Received: from cygnus.com (localhost [127.0.0.1])
	by localhost.cygnus.com (Postfix) with ESMTP
	id 069CB3CC9; Thu, 10 Jan 2002 10:15:28 -0500 (EST)
Message-ID: <3C3DB00F.6050605@cygnus.com>
Date: Thu, 10 Jan 2002 07:15:00 -0000
From: Andrew Cagney <ac131313@cygnus.com>
User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-US; rv:0.9.7) Gecko/20020103
X-Accept-Language: en-us
MIME-Version: 1.0
To: Nick Clifton <nickc@cambridge.redhat.com>
Cc: fnf@redhat.com, gdb-patches@sources.redhat.com
Subject: Re: RFC: ARM simulator coredump
References: <200201092258.g09MwlU26601@fred.ninemoons.com>	<3C3CD2BA.30209@cygnus.com> <m3n0zma1sm.fsf@north-pole.nickc.cambridge.redhat.com>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
X-SW-Source: 2002-01/txt/msg00225.txt.bz2

> +      {
>> +	ARMword temp;
> 
> 
> There is no need to declare a 'temp' variable here.  There is a
> variable of the same type and name declared at the top of the
> function, and it is no longer being used by this point.


Er, perhaphs here.  In general, keeping declarations as local as 
possible is a good thing :-)

enjoy,
Andrew