Mailing-List: contact lucy-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: lucy-dev@incubator.apache.org
Received-SPF: neutral (nike.apache.org: local policy)
MIME-Version: 1.0
In-Reply-To: <20111103025242.GB26068@rectangular.com>
References: <4E9F98AC.7050602@peknet.com>
 <alpine.DEB.2.00.1111010959110.17088@bester>
 <20111103025242.GB26068@rectangular.com>
From: Nathan Kurz <nate@verse.com>
Date: Wed, 2 Nov 2011 20:51:46 -0700
Message-ID: 
 <CAFAN8vzSGE47RZqgCgWUn+HXpM_iaf42-OHKRVRqD0+0yBxRUQ@mail.gmail.com>
To: lucy-dev@incubator.apache.org
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [lucy-dev] [VOTE] Apache Lucy (incubating) 0.2.2 RC 1

On Wed, Nov 2, 2011 at 7:52 PM, Marvin Humphrey <marvin@rectangular.com> wr=
ote:
> Here's the line in question:
>
> =C2=A0 =C2=A0fprintf(stderr, "Can't malloc %" U64P " bytes.\n", (uint64_t=
)count);
>
> Thanks for pointing these warnings out, Hoss. =C2=A0I would like to silen=
ce them,
> but it's hard to do that portably -- for instance, we can't cast to "long=
 long
> int" because it's not there on all systems.

I do believe that we should compile warning free, but I'm not sure I'm
understanding the issue here.  Hoss's error is on GCC, we're
targetting C99, and you already have U64P defined as a macro?

Wouldn't "%zu" work fine without the cast?  I think it's standard C99
at this point, and not just a GCC extension.  But if you've got a
macro anyway, it should be fine to use.  Historically, the reason to
avoid it was support under Windows, where you would have to use "%Iu"
(that's a capital 'i', not a lower case 'L').

http://msdn.microsoft.com/en-us/library/tcxf1dw6(v=3DVS.100).aspx

Or am I missing another layer of complexity?

--nate