[Ocfs2-devel] Request to remove -Wfno-format
Rusty Lynch
rusty at linux.co.intel.com
Fri Feb 13 14:24:27 CST 2004
On Fri, Feb 13, 2004 at 02:15:37PM -0800, Manish Singh wrote:
> On Thu, Feb 12, 2004 at 09:43:30PM -0800, Rusty Lynch wrote:
> > On Thu, Feb 12, 2004 at 07:44:34PM -0800, Manish Singh wrote:
> > > On Thu, Feb 12, 2004 at 05:52:01PM -0800, Rusty Lynch wrote:
> > > > (There are a couple of places where I cast a size_t to int to stop warnings
> > > > on 64bit machines... can't think of a cleaner way of printing size_t since
> > > > prink doesn't directly handle the type.)
> > >
> > > %Zu works for size_t.
> >
> > Ah.. I see that now in the fprintf man page.
> >
> > Here is another version of the patch that uses %zu where appropriate.
>
> Well, I really meant "%Zu". 'z' is a modifier that was introduced in C99,
> but it's only in very recent 2.4.x printk implementations, i.e. none of
> current AS 2.1, EL3, or UL have it, which means we can't use it. 'Z' is
> GNU extension (that's deprecated now in favor of the C99 modifier) that
> *is* in all the printks we need to support, so we should use that.
> __attribute__ format should know about both anyway.
>
ah... I'm not using any older Linux installs and didn't realize 'z' was
such a recent addition. I will change the patch over the 'Z'.
> BTW, are you planning on replacing HI/LO with %L in a later patch?
I'll go ahead and do this in one chunk before I disapear for a long weekend.
--rusty
More information about the Ocfs2-devel
mailing list