[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget
From: |
Jure Grabnar |
Subject: |
Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget |
Date: |
Wed, 19 Mar 2014 19:40:57 +0100 |
Hi,
my proposal is available on
http://www.google-melange.com/gsoc/proposal/public/google/gsoc2014/toomanysecrets/5629499534213120
I'm looking forward to your review.
Regards,
Jure Grabnar
On 18 March 2014 01:06, Darshit Shah <address@hidden> wrote:
> Hi Jure,
>
> Thanks for your patches. However, I do have a few comments about the same:
>
> 1. Trailing Whitespaces: This is essentially extra whitespaces at the
> end of a line or on a blank line. See [1] and [2] for more
> information.
> 2. The indentation is mostly right, but sometimes off.
> 3. Your first patch is missing a ChangeLog. Every commit must be
> accompanied by a ChangeLog entry, no matter how trivial it is.
> 4. Your 2nd patch seems to revert things from the first one. This
> usually means some cleanup is needed.
>
> I'm not completely sure of some of the details of the lines you change
> in your second patch, but they seem a little sketchy. I'll have to dig
> into the code and check it out.
>
> Also, for a non-trivial (>10 LoC) patch, you'll first need to submit
> your copyright assignment to the FSF.
> Giuseppe will arrange for the documents as soon as your patch is ready.
>
> The code however, does fix a segfault and maybe a few compiler
> warnings. When it fixes something, an explanation is usually a nice
> idea.
>
> [1]
> http://codeimpossible.com/2012/04/02/Trailing-whitespace-is-evil-Don-t-commit-evil-into-your-repo-/
> [2]
> https://stackoverflow.com/questions/1583406/why-does-git-care-about-trailing-whitespace-in-my-files
>
> On Mon, Mar 17, 2014 at 6:56 PM, Jure Grabnar <address@hidden> wrote:
> > Hi,
> >
> > this patch fixes some of compiler warnings. I was uncertain for the
> > remaining ones (5) - I believe some of them might be stubs for upcoming
> > features.
> >
> > Best Regards,
> >
> >
> > Jure Grabnar (toomanysecrets)
>
>
>
> --
> Thanking You,
> Darshit Shah
>
- [Bug-wget] [GSoC] Extend concurrency support in Wget, Jure Grabnar, 2014/03/17
- [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Jure Grabnar, 2014/03/17
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Darshit Shah, 2014/03/17
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget,
Jure Grabnar <=
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Darshit Shah, 2014/03/20
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Jure Grabnar, 2014/03/20
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Darshit Shah, 2014/03/21
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Yousong Zhou, 2014/03/21
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Jure Grabnar, 2014/03/22
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Yousong Zhou, 2014/03/22
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Darshit Shah, 2014/03/27
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Jure Grabnar, 2014/03/27
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Jure Grabnar, 2014/03/28
- Re: [Bug-wget] Fwd: [GSoC] Extend concurrency support in Wget, Jure Grabnar, 2014/03/28