From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60082) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZzWmE-00056q-LA for qemu-devel@nongnu.org; Thu, 19 Nov 2015 16:29:59 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZzWmB-0007Yf-EV for qemu-devel@nongnu.org; Thu, 19 Nov 2015 16:29:58 -0500 Received: from mx1.redhat.com ([209.132.183.28]:34275) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZzWmB-0007YZ-7B for qemu-devel@nongnu.org; Thu, 19 Nov 2015 16:29:55 -0500 References: <1447936168-9808-1-git-send-email-armbru@redhat.com> <20151119130339.GC2653@work-vm> <20151119131047.GF9247@redhat.com> From: Eric Blake Message-ID: <564E3F47.3030209@redhat.com> Date: Thu, 19 Nov 2015 14:29:43 -0700 MIME-Version: 1.0 In-Reply-To: <20151119131047.GF9247@redhat.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="IrPnGtckiFQ6t8dm0rtT5mJj8p94IRoxh" Subject: Re: [Qemu-devel] [PATCH] typedefs: Put them back into alphabetical order List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Daniel P. Berrange" , "Dr. David Alan Gilbert" Cc: qemu-devel@nongnu.org, qemu-trivial@nongnu.org.com, Markus Armbruster This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --IrPnGtckiFQ6t8dm0rtT5mJj8p94IRoxh Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 11/19/2015 06:10 AM, Daniel P. Berrange wrote: > On Thu, Nov 19, 2015 at 01:03:39PM +0000, Dr. David Alan Gilbert wrote:= >> * Markus Armbruster (armbru@redhat.com) wrote: >>> "Please keep this list in alphabetical order" has been more honoured >>> in the breach than in the observance. Clean up. Whose alphabet? ... >>> >>> While there, drop a redundant struct declaration. >>> >>> Signed-off-by: Markus Armbruster >> Reviewed-by: Eric Blake >> Hmph, it's only a year since I sorted this file. >> At least one of these changes is from a misordering within one patch! >=20 > Which just goes to show that any such rules about sorting are > not worth the hassle unless you also provide some make rule to > validate them as part of 'make test'. It shouldn't be too > hard to hook it up - a make rule that runs this: >=20 > grep typedef include/qemu/typedefs.h > orig.txt > sort orig.txt > sorted.txt =2E..This would have to be 'LC_ALL=3DC sort orig.txt ...' to ensure that difference in locale doesn't change the order (whether due to case-insensitivity, or due to ignoring whitespace). Picking on one of the changes in the commit: $ printf 'typedef struct PropertyInfo PropertyInfo;\ntypedef struct Property Property;\n' | sort # LANG=3Den_US.UTF-8 typedef struct PropertyInfo PropertyInfo; typedef struct Property Property; $ printf 'typedef struct PropertyInfo PropertyInfo;\ntypedef struct Property Property;\n' | LC_ALL=3DC sort typedef struct Property Property; typedef struct PropertyInfo PropertyInfo; > diff orig.txt sorted.txt > /dev/null >=20 > as part of 'make test' would suffice Also, Markus probably noticed the problem because I have a pending patch that will be adding QObject to the list, and my addition will conflict since it falls next to the fixup for Range vs. QEMUTimerListGroup. Whichever one goes in first gets to watch the other rebase :) --=20 Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org --IrPnGtckiFQ6t8dm0rtT5mJj8p94IRoxh Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Public key at http://people.redhat.com/eblake/eblake.gpg Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQEcBAEBCAAGBQJWTj9IAAoJEKeha0olJ0NqfoUH/j67CHIYaNScCIXJcCbIJtwq j0M/7Fc5LARkOYQI5jxkT9M8LFGYLq8yD0V+3OhZ3hY9r3z5BHwUOAwqm/pr8L1s S3XIVERr1vMYmjRjyWTV/VseTKZorJ82CkFc6p/ivNAgDBsp7gqjfN2fPSp+aPON D8enchZaOrWauSeOBUr7RKua78pCs8ql6VNWAbplXd78/QkxdgyFhTWraBnDgSrX FtNOA/mn+TlF5OcX5xo+ZJlvZ2ZFdSn84RrIJCXFwPhXS2yzTXIvVRFROafOqef0 Zvshzj3sfoAGnqtgsBabAVexaSAxqfPp/uskH5ETlG4tLkjh1ZYvbCYSuHJL24E= =qaH3 -----END PGP SIGNATURE----- --IrPnGtckiFQ6t8dm0rtT5mJj8p94IRoxh--