From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 20529 invoked by alias); 5 Oct 2008 18:40:47 -0000 Received: (qmail 20521 invoked by uid 22791); 5 Oct 2008 18:40:46 -0000 X-Spam-Check-By: sourceware.org Received: from sibelius.xs4all.nl (HELO sibelius.xs4all.nl) (82.92.89.47) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sun, 05 Oct 2008 18:40:11 +0000 Received: from brahms.sibelius.xs4all.nl (kettenis@localhost.sibelius.xs4all.nl [127.0.0.1]) by brahms.sibelius.xs4all.nl (8.14.3/8.14.3) with ESMTP id m95Ie3V1018403; Sun, 5 Oct 2008 20:40:04 +0200 (CEST) Received: (from kettenis@localhost) by brahms.sibelius.xs4all.nl (8.14.3/8.14.3/Submit) id m95Ie3pD009409; Sun, 5 Oct 2008 20:40:03 +0200 (CEST) Date: Sun, 05 Oct 2008 18:40:00 -0000 Message-Id: <200810051840.m95Ie3pD009409@brahms.sibelius.xs4all.nl> From: Mark Kettenis To: msnyder@vmware.com CC: msnyder@vmware.com, mark@klomp.org, gdb-patches@sourceware.org, teawater@gmail.com In-reply-to: <48E906B5.8070600@vmware.com> (message from Michael Snyder on Sun, 05 Oct 2008 11:25:57 -0700) Subject: Re: [RFA record/replay] cast to avoid compiler warning References: <48E7B927.9050207@vmware.com> <09B97D2C4B49DF409E2D018999AC6FE115B42A89C4@PA-EXMBX14.vmware.com> <20081005160411.GA29968@caradoc.them.org> <48E906B5.8070600@vmware.com> Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2008-10/txt/msg00136.txt.bz2 > Date: Sun, 05 Oct 2008 11:25:57 -0700 > From: Michael Snyder > > Daniel Jacobowitz wrote: > > On Sun, Oct 05, 2008 at 08:44:38AM -0700, Michael Snyder wrote: > >> Yes, but we don't know that gdb will always be built using gnulib. > > > > Gnulib is a library of portability replacement routines that GDB can > > import. We already use it for stdint.h and memchr / memmem. We could > > easily add it for further routines. > > OK. So, would you favor using the "%zd" modifier instead of a cast? > > I mainly didn't do that because there were no existing instances > of it in gdb now, but I was just being conservative... We still only require ISO C90 to build GDB, so we shouldn't use %zd.