From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Mahoney Subject: Re: [PATCH 2/9] sector_t format string Date: Fri, 11 Aug 2006 10:48:21 -0400 Message-ID: <44DC98B5.4000602@suse.com> 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> <44DB945F.5080102@suse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: Andrew Morton , ext2-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, cmm@us.ibm.com, linux-fsdevel@vger.kernel.org, Alexey Dobriyan Return-path: To: Jan Engelhardt In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: ext2-devel-bounces@lists.sourceforge.net Errors-To: ext2-devel-bounces@lists.sourceforge.net List-Id: linux-fsdevel.vger.kernel.org -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Jan Engelhardt wrote: >>> Will >>> >>> printk("%S", sector_t); >>> >>> kill at least one kitten? >> I like the general idea. I think that having to cast every time you want >> to print a sector number is pretty gross. I had something more like %Su >> in mind, though. > > What will happen if you run out of %[a-z] ? Are we really expecting that many global structure members to be variable width? I only propose adding another option because whenever a sector is printed, it must be casted to avoid warnings. Other replies commented on how gcc won't recognize the new option, so we'd receive warnings anyway. Cleaner code that causes warnings doesn't sound like a big win after all. - -Jeff - -- Jeff Mahoney SUSE Labs -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org iD8DBQFE3Ji1LPWxlyuTD7IRApAEAJ9ApkoyKwmTReZindjJmkuU/0yhbACgk0Uu zG8eXN3RzU1wKFVrRlr3xO8= =e6D2 -----END PGP SIGNATURE----- ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642