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 1/2] Use write_message instead of write

From: "Gabriel F. T. Gomes" <gftg at linux dot vnet dot ibm dot com>
To: Florian Weimer <fweimer at redhat dot com>
Cc: libc-alpha at sourceware dot org
Date: Mon, 7 Nov 2016 22:40:01 -0200
Subject: Re: [PATCH 1/2] Use write_message instead of write
Authentication-results: sourceware.org; auth=none
References: <1477961028-12150-1-git-send-email-gftg@linux.vnet.ibm.com> <1477961028-12150-2-git-send-email-gftg@linux.vnet.ibm.com> <0947da13-c81e-bac5-df89-465cdecc70d6@redhat.com>

On Mon, 7 Nov 2016 18:33:25 +0100
Florian Weimer <fweimer@redhat.com> wrote:

> On 11/01/2016 01:43 AM, Gabriel F. T. Gomes wrote:
> > In the test cases, there are writes to stdout which do not check the result
> > value.  This patch replaces such occurrences with calls to write_message,
> > which properly deals with the unused result.
> >
> > 2016-10-28  Gabriel F. T. Gomes  <gftg@linux.vnet.ibm.com>
> >
> > 	* nptl/tst-kill3.c (handler): Use write_message in place of write.
> > 	* nptl/tst-kill6.c (handler): Likewise.
> > 	* nptl/tst-tls2.c (CB): Likewise.
> > 	* nptl/tst-tls3.c (CB): Likewise.  
> 
> Looks good to me.

Thanks.  Pushed as d5b38790.

References:
- [PATCH 0/2] Cleanup some calls to write with unused result
  - From: Gabriel F. T. Gomes
- [PATCH 1/2] Use write_message instead of write
  - From: Gabriel F. T. Gomes
- Re: [PATCH 1/2] Use write_message instead of write
  - From: Florian Weimer

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