[parted-devel] [PATCH] next branch

Otavio Salvador otavio at ossystems.com.br
Wed Jun 3 13:27:31 UTC 2009


Hello Joel,

On Wed, Jun 3, 2009 at 9:15 AM, Joel Granados <jgranado at redhat.com> wrote:
> In summary, and to clarify my stand on this issue:
> I agree that there might be "cons" regarding individual function
> comments in the commit log.
>  a. Make the developer think that the function doc is already written
>  b. The commit comment is not in sync with the commit.
>  c. The specific commit function comment repeats stuff that is said in the main
>    commit message.
>  d. The commit function comment becomes extensive and gives too much info.
>
> But, if the commit function comments don't fit in (a-d), and they are
> sound, and they actually add some context when reading the log, I would
> accept the patch.

I fully agree. It can be of great value for people finding by
information in log but, as coding standards, we will need to use good
sense as a metric in final decision.

-- 
Otavio Salvador                  O.S. Systems
E-mail: otavio at ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854         http://projetos.ossystems.com.br



More information about the parted-devel mailing list