This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH v3 4/7] Add __v*printf_internal with flags arguments

From: Adhemerval Zanella <adhemerval dot zanella at linaro dot org>
To: libc-alpha at sourceware dot org
Date: Thu, 22 Nov 2018 15:59:25 -0200
Subject: Re: [PATCH v3 4/7] Add __v*printf_internal with flags arguments
References: <20181115214449.19262-1-gabriel@inconstante.eti.br> <20181115214449.19262-5-gabriel@inconstante.eti.br> <80aca9c1-0192-38d9-9f08-70c5765fa1f8@linaro.org>


On 22/11/2018 15:55, Adhemerval Zanella wrote:
> 
> 
> On 15/11/2018 19:44, Gabriel F. T. Gomes wrote:
>> From: Zack Weinberg <zackw@panix.com>
>>
>> Changes since v2:
>>
>>   - Expanded the comments about PRINTF_LDBL_IS_DBL and PRINTF_FORTIFY.
>>   - Removed the parameter-less macros LDBL_IS_DBL and DO_FORTIFY.
>>     Explicitly compare the flags.
> 
> LGTM, thanks.
And, as before please add a note on CL that it fixes BZ#11319 by unifying both 
fortified and non-fortified vdprintf initialization.

References:
- [PATCH v3 0/7] Use more flags parameters instead of global bits in stdio
  - From: Gabriel F. T. Gomes
- [PATCH v3 4/7] Add __v*printf_internal with flags arguments
  - From: Gabriel F. T. Gomes
- Re: [PATCH v3 4/7] Add __v*printf_internal with flags arguments
  - From: Adhemerval Zanella

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]