From patchwork Mon Dec 15 15:01:47 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stefan Liebler X-Patchwork-Id: 4255 Received: (qmail 5831 invoked by alias); 15 Dec 2014 15:03:00 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 5780 invoked by uid 89); 15 Dec 2014 15:02:59 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_LOW, SPF_HELO_PASS, SPF_PASS, T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: plane.gmane.org To: libc-alpha@sourceware.org From: Stefan Liebler Subject: Re: [PATCH] S390: Get rid of Date: Mon, 15 Dec 2014 16:01:47 +0100 Lines: 44 Message-ID: References: <20141212211053.0757C2C3ADB@topped-with-meat.com> Mime-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 In-Reply-To: <20141212211053.0757C2C3ADB@topped-with-meat.com> On 12/12/2014 10:10 PM, Roland McGrath wrote: > I think %td is the right format for ptrdiff_t. > Yes you are right. Here is the new patch version, which uses %td instead of %Zd. Ok to commit? --- 2014-12-15 Stefan Liebler * libio/tst-widetext.c (do_test): Use format type %td instead of %Zd for ptrdiff_t in order to avoid format warning. diff --git a/libio/tst-widetext.c b/libio/tst-widetext.c index acab72b..83345d7 100644 --- a/libio/tst-widetext.c +++ b/libio/tst-widetext.c @@ -291,7 +291,7 @@ do_test (void) { if (fgetws (wcp, &wc2buf[wcsize] - wcp + 1, fp) == NULL) { - printf ("%u: short read using fgetws (only %Zd of %Zd)\n", + printf ("%u: short read using fgetws (only %td of %Zd)\n", __LINE__, wcp - wc2buf, wcsize); status = 1; break;