Date: Thu, 10 Aug 2006 12:57:47 -0700
From: Andrew Morton <akpm@osdl.org>
To: Alexey Dobriyan <adobriyan@gmail.com>
Cc: cmm@us.ibm.com, linux-kernel@vger.kernel.org,
       ext2-devel@lists.sourceforge.net, linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH 2/9] sector_t format string
Message-Id: <20060810125747.87f7b1dc.akpm@osdl.org>
In-Reply-To: <20060810194440.GA6845@martell.zuzino.mipt.ru>
References: <1155172843.3161.81.camel@localhost.localdomain>
	<20060809234019.c8a730e3.akpm@osdl.org>
	<20060810191747.GL20581@ca-server1.us.oracle.com>
	<20060810194440.GA6845@martell.zuzino.mipt.ru>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1206
Lines: 33

On Thu, 10 Aug 2006 23:44:40 +0400
Alexey Dobriyan <adobriyan@gmail.com> wrote:

> On Thu, Aug 10, 2006 at 12:17:47PM -0700, Joel Becker wrote:
> > On Wed, Aug 09, 2006 at 11:40:19PM -0700, Andrew Morton wrote:
> > > On Wed, 09 Aug 2006 18:20:43 -0700
> > > Mingming Cao <cmm@us.ibm.com> wrote:
> > > 
> > > > Define SECTOR_FMT to print sector_t in proper format
> > > 
> > > We've thus-far avoided doing this.  In fact a similar construct in
> > > device-mapper was recently removed.
> > 
> > 	Yeah, OCFS2 had similar formats, and we were asked to change
> > them to naked casts before inclusion.  Seems quite consistent with the
> > rest of the kernel.
> 
> Will
> 
> 	printk("%S", sector_t);
> 
> kill at least one kitten?

It would be really nice to be able to define local enhancements like this
to printf.  It would solve lots of these problems quite nicely.

Bus alas, there's no way (afaik) to teach __attribute__((format)) about
them, so gcc will warn.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/