Re: [PATCH] savedir: optionally produce ordered directory list

2014-02-11 Thread Paul Eggert
Thanks for doing that. How about the attached (untested) patch instead? It simplifies things by removing the obsolescent function fdsavedir, and omitting the static variable that stores internal state. The downside is that it changes the API, but that's OK in gnulib. It fixes a couple of ot

[PATCH] savedir: optionally produce ordered directory list

2014-02-11 Thread Sergey Poznyakoff
Hello, According to [1], using directory lists ordered by inode can speed up tar archivation by up to 40% on ext[34] filesystems (confirmed by my own measurements). Please take a look at the attached patch. Is it OK to apply? Regards, Sergey [1] https://savannah.gnu.org/patch/?7892 >From 92dd