This is the mail archive of the
cygwin-patches
mailing list for the Cygwin project.
Re: Cygwin Filesystem Performance degradation 1.7.5 vs 1.7.7, and methods for improving performance
- From: Christopher Faylor <cgf-use-the-mailinglist-please at cygwin dot com>
- To: cygwin-patches at cygwin dot com
- Date: Mon, 6 Sep 2010 11:25:46 -0400
- Subject: Re: Cygwin Filesystem Performance degradation 1.7.5 vs 1.7.7, and methods for improving performance
- References: <4C84B9EF.9030109@gmail.com> <20100906132409.GB14327@calimero.vinschen.de>
- Reply-to: cygwin-patches at cygwin dot com
Thanks for the patch and for all of the work you put into it.
On Mon, Sep 06, 2010 at 03:24:09PM +0200, Corinna Vinschen wrote:
>The patch is also missing a ChangeLog entry. I only took a quick glance
>over the patch itself. The code doesn't look correctly formatted in GNU
>style. Also, using the diff -up flags would be helpful.
And, this is the type of patch which would be better served if submitted
in small chunks. You have multiple changes in your 1158 line patch and
they don't seem to all be interrelated.
Also, in addition to formatting concerns, you don't seem to have used
comments very much. Corinna and I have been making a concerted effort
to comment changes more thoroughly so it would be nice if your patch
contained more of those.
I didn't look at the patch very closely either since there are copyright
issues but some of your conclusions don't seem right to me. I agree
with Corinna's response.
cgf