From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-18813-listarch-gdb-patches=sourceware.cygnus.com@sources.redhat.com>
Received: (qmail 29594 invoked by alias); 19 Sep 2002 15:29:08 -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 29586 invoked from network); 19 Sep 2002 15:29:05 -0000
Received: from unknown (HELO localhost.redhat.com) (216.138.202.10)
  by sources.redhat.com with SMTP; 19 Sep 2002 15:29:05 -0000
Received: from ges.redhat.com (localhost [127.0.0.1])
	by localhost.redhat.com (Postfix) with ESMTP
	id AEE163DB8; Thu, 19 Sep 2002 11:29:02 -0400 (EDT)
Message-ID: <3D89ED3E.9060106@ges.redhat.com>
Date: Thu, 19 Sep 2002 08:29:00 -0000
From: Andrew Cagney <ac131313@ges.redhat.com>
User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-US; rv:1.0.0) Gecko/20020824
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: Eli Zaretskii <eliz@is.elta.co.il>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [rfa:doco] document frame_align()
References: <Pine.SUN.3.91.1020919145739.2289F-100000@is>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
X-SW-Source: 2002-09/txt/msg00487.txt.bz2

> On Wed, 18 Sep 2002, Andrew Cagney wrote:
> 
> 
>> This documents the frame_align() architecture method.  It also clarifies 
>> the behavior of stack_align().
>> 
>> ok?
> 
> 
> Yes.  However:
> 
>   Unlike @ref{STACK_ALIGN}, this function always adjusts the address
> 
> Please don't use this technique.  It sounds cute at first glance, but 
> produces text in the Info version which looks awkwardly:
> 
>   Unlike *note STACK_ALIGN, this function always adjusts the address
> 
> Instead, do it more straightforwardly:
> 
>   Unlike @code{STACK_ALIGN} (@pxref{STACK_ALIGN}), ...

Ok.

> (In this particular case, the @pxref is actually redundant, since you 
> have a @pxref like that a few lines before that.)

Hmm, ok.  Gone.  Committed with just ``Unlike @code{STACK_ALIGN} ...''.

thanks,
Andrew