Date: Sun, 1 Dec 2002 09:21:59 +0200 (IST)
From: Eli Zaretskii <eliz AT is DOT elta DOT co DOT il>
X-Sender: eliz AT is
To: Richard Dawe <rich AT phekda DOT freeserve DOT co DOT uk>
cc: djgpp-workers AT delorie DOT com
Subject: Re: printf and the 'hh' conversion specifier (C99) [PATCH]
In-Reply-To: <E18IAGz-0000S3-00@phekda.freeserve.co.uk>
Message-ID: <Pine.SUN.3.91.1021201092042.17170A-100000@is>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Reply-To: djgpp-workers AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: djgpp-workers AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk


On Sat, 30 Nov 2002, Richard Dawe wrote:

> Below is a patch to add support for the 'hh' conversion specifier.

Thanks!

> This is shorter than a short - a char. The only change that concerns
> me is:
> 
>     ARG(int);
> 
> switched to:
> 
>     ARG(signed);
> 
> I'm pretty sure this is equivalent.

Yes, I think so, since our `int' is a signed data type.

> It seems to work OK - I ran
> the Cygnus test suite (*) - tests/cygnus - and the results were
> unaffected by this change.
> 
> (*) Eli suggested the Cygnus test suite as being hard on *scanf()
>     and *printf() previously.

Yes, that's a good idea to test any changes to doprint.c