From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-63823-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 31020 invoked by alias); 19 May 2009 02:34:35 -0000
Received: (qmail 31010 invoked by uid 22791); 19 May 2009 02:34:34 -0000
X-SWARE-Spam-Status: No, hits=-1.7 required=5.0 	tests=AWL,BAYES_00,SARE_MSGID_LONG40,SPF_PASS
X-Spam-Check-By: sourceware.org
Received: from ti-out-0910.google.com (HELO ti-out-0910.google.com) (209.85.142.187)     by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 19 May 2009 02:34:25 +0000
Received: by ti-out-0910.google.com with SMTP id a1so330738tib.12         for <gdb-patches@sourceware.org>; Mon, 18 May 2009 19:34:21 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.110.69.14 with SMTP id r14mr455504tia.59.1242700461718; Mon,  	18 May 2009 19:34:21 -0700 (PDT)
In-Reply-To: <20090518153510.GA23052@ednor.casa.cgf.cx>
References: <daef60380905101031s1fd37293o1358d83357032492@mail.gmail.com> 	 <20090510174809.GA25909@ednor.casa.cgf.cx> 	 <daef60380905101107u25e55c38jaf05c7977adf57b6@mail.gmail.com> 	 <20090510235109.GD25909@ednor.casa.cgf.cx> 	 <daef60380905180143l63bcc007n9821638179ef84ec@mail.gmail.com> 	 <20090518153510.GA23052@ednor.casa.cgf.cx>
Date: Tue, 19 May 2009 02:34:00 -0000
Message-ID: <daef60380905181934j7ec54c1aj4f88a6e06387abfa@mail.gmail.com>
Subject: Re: [Prec/RFA] fix build error of prec in cygwin
From: Hui Zhu <teawater@gmail.com>
To: gdb-patches ml <gdb-patches@sourceware.org>, Hui Zhu <teawater@gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
X-IsSubscribed: yes
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2009-05/txt/msg00386.txt.bz2

On Mon, May 18, 2009 at 23:35, Christopher Faylor
<cgf-use-the-mailinglist-please@sourceware.org> wrote:
> On Mon, May 18, 2009 at 04:43:36PM +0800, Hui Zhu wrote:
>>On Mon, May 11, 2009 at 07:51, Christopher Faylor wrote:
>>> So wouldn't that indicate that your format specifier is wrong and should
>>> be "%u" rather than "%d"? ?Coercion is something that should be avoided
>>> unless it is absolutely necessary.
>>>
>>
>>I changed it.
>
> It looks like this diff is against your previous version rather than CVS
> so it is confusing. =A0But, it still seems like you're still coercing
> using (int)len but now you're using %u. =A0So, it looks like you've just
> swapped the problem.

No, it based on cvs-head. Are you really try?

With out (int) it will get:
../../src/gdb/linux-record.c: In function `record_linux_system_call':
../../src/gdb/linux-record.c:1787: warning: unsigned int format, long
int arg (arg 4)
make: *** [linux-record.o] Error 1

In Cygwin.

Thanks,
Hui