From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1MU4iK-00006x-3m for mharc-grub-devel@gnu.org; Thu, 23 Jul 2009 16:16:28 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MU4iE-0008VG-Bh for grub-devel@gnu.org; Thu, 23 Jul 2009 16:16:22 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MU4i6-0008RW-Vt for grub-devel@gnu.org; Thu, 23 Jul 2009 16:16:20 -0400 Received: from [199.232.76.173] (port=46500 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MU4i6-0008RM-SO for grub-devel@gnu.org; Thu, 23 Jul 2009 16:16:14 -0400 Received: from c60.cesmail.net ([216.154.195.49]:4324) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_ARCFOUR_SHA1:16) (Exim 4.60) (envelope-from ) id 1MU4i5-0000UD-Mr for grub-devel@gnu.org; Thu, 23 Jul 2009 16:16:14 -0400 Received: from unknown (HELO smtprelay2.cesmail.net) ([192.168.1.112]) by c60.cesmail.net with ESMTP; 23 Jul 2009 16:16:07 -0400 Received: from [192.168.0.22] (static-72-92-88-10.phlapa.fios.verizon.net [72.92.88.10]) by smtprelay2.cesmail.net (Postfix) with ESMTPSA id D129B34C6A for ; Thu, 23 Jul 2009 16:26:55 -0400 (EDT) From: Pavel Roskin To: The development of GRUB 2 In-Reply-To: References: <1248289850.10302.57.camel@accesodirecto.casa> <1248375298.4660.174.camel@accesodirecto.casa> Content-Type: text/plain; charset="ISO-8859-1" Date: Thu, 23 Jul 2009 16:16:08 -0400 Message-Id: <1248380168.2661.71.camel@mj> Mime-Version: 1.0 X-Mailer: Evolution 2.26.3 (2.26.3-1.fc11) Content-Transfer-Encoding: 8bit X-detected-operating-system: by monty-python.gnu.org: Genre and OS details not recognized. Subject: Re: [RFC, PATCH] C99 format specifiers for fixed-length integer types X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: The development of GRUB 2 List-Id: The development of GRUB 2 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 23 Jul 2009 20:16:23 -0000 On Thu, 2009-07-23 at 21:51 +0200, Vladimir 'phcoder' Serbinenko wrote: > 2009/7/23 Javier Martín : > > Here is a new version which also incorporates the C99 integer constant > > macros. To avoid excess verbosity, all macros have now names like Gx, > > where x is the standard name. Thus, PRIx64, UINT64_C(a) --> GPRIx64, > > GUINT64_C(a). > Please respect current convention of using GRUB_ as prefix for macros > Are the macros *_C really useful? Anyway it's to be discussed > separately. Could you not do increase previous patch bt make a new one > to separate what is already in discussion from new things. > @Pavel: do you have any further comments? I believe it's not worth the trouble at this point. There are many patches that I make and never send or never commit because I'm not sure that the change is valuable enough. Every change comes with a risk of breaking something. For instance, I planned to remove "cli" in the bootloader, but decided that it's not worth the risk. I wanted to remove support for module unloading, but decided that there is a downside for users, and there are better ways to reduce the size of core.img. I wanted to simplify awk invocation, but decided not to pursue it at this time, as it would conflict with other efforts to reorganize the build system, and the benefits would be mostly theoretical. I think a good developer should be able to drop changes that are not exactly useful and move on. We have other issues. Javier mentioned -Wconversion. It makes the compiler very noisy, but some issues it found are real, such as the problem with ALIGN_UP. It's more important than pushing the same patch. One day we may find a nice solution to the issue of file formats. Maybe a new C standard appears with new format specifications. Maybe gcc will add some checks. But as it stands now, we won't benefit from it enough to justify less readable code. Let's move on and work on the issues where we can make the difference now. -- Regards, Pavel Roskin