[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] maint: sort private sym lists

On 10/21/2010 02:15 AM, Daniel Veillard wrote:
On Wed, Oct 20, 2010 at 11:47:59AM -0600, Eric Blake wrote:
* src/libvirt_private.syms: Sort by header name, then within
header, and drop duplicate virNetworkDefParseNode,
virFileLinkPointsTo and virXPathBoolean.

Fixes the duplication first noted here, and hopefully makes the
file easier to maintain.

  src/libvirt_private.syms |  746 +++++++++++++++++++++++-----------------------
  1 files changed, 376 insertions(+), 370 deletions(-)


Thanks; pushed. I also added a comment at the top of the file reminding about the sort order.

I wonder if it's worth trying to enforce that sorting, probably not,

I started on that yesterday, but gave up after getting to the point where I could reliably sort the file, but not refactor the sorted output back into the original form (maybe it's just me, but sed's x and G operators are not the most common, and I had to read the manual):

$ sed '/^$/,$ { # For lines after the header...
 /^#/ { x; s/^/ /; }; # Stick the filename in hold space
 /^[^#]/ { G; s/\n/ /; }; # For all other lines, append the filename
 /^$/ { x; d; } # and clear the hold space after each file
}' src/libvirt_private.syms | sort -k 3,3 -k 1,1

Does anyone wants to take that idea further to automate keeping things sorted? For now, I'm fine with just manual efforts.

Eric Blake   eblake redhat com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]