[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] typedefs: Put them back into alphabetical order
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH] typedefs: Put them back into alphabetical order |
Date: |
Thu, 19 Nov 2015 13:10:47 +0000 |
User-agent: |
Mutt/1.5.23 (2015-06-09) |
On Thu, Nov 19, 2015 at 01:03:39PM +0000, Dr. David Alan Gilbert wrote:
> * Markus Armbruster (address@hidden) wrote:
> > "Please keep this list in alphabetical order" has been more honoured
> > in the breach than in the observance. Clean up.
> >
> > While there, drop a redundant struct declaration.
> >
> > Signed-off-by: Markus Armbruster <address@hidden>
>
> Hmph, it's only a year since I sorted this file.
> At least one of these changes is from a misordering within one patch!
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:
grep typedef include/qemu/typedefs.h > orig.txt
sort orig.txt > sorted.txt
diff orig.txt sorted.txt > /dev/null
as part of 'make test' would suffice
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|