|
From: | Eduardo |
Subject: | Re: Feedback about a patch. |
Date: | Mon, 17 Mar 2014 00:22:35 +0100 |
I took a quick look and noted a couple of problems.On Mar 16, 2014, at 5:19 PM, Eduardo <address@hidden> wrote:
> Hello, anyone familiarized with the process of generating and submitting a patch could please give me some feedback about this one I submitted some days ago? It would be so helpful for me to know if I have followed correctly the guidelines, coding style, etc. I want to submit another patch but I don't know if I did well the first time.
> Thanks in advance.
>
> Link: https://savannah.gnu.org/bugs/?41839
>
> Eduardo (edu159)
(1) The ChangeLog shouldn't include your user-name/email (i.e. do not add "user: Eduardo Ramos (edu159) <address@hidden>"). This is already present in the mercurial header information.
(2) Comments should begin with hash-hash-space (i.e. "## "). A few of yours only have one hash, and some of these don't have the space.
These are minor points and can easily be fixed using an editor when the operation of the patch is verified and then pushed. I notice you've included several tests. That will make verifying everything works as expected straightforward.
Ben
[Prev in Thread] | Current Thread | [Next in Thread] |