[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Fix 155. (issue 5032047)
From: |
Reinhold Kainhofer |
Subject: |
Re: Fix 155. (issue 5032047) |
Date: |
Sat, 17 Sep 2011 14:36:04 +0200 |
User-agent: |
KMail/1.13.6 (Linux/2.6.38-11-generic; KDE/4.7.0; i686; ; ) |
Am Saturday, 17. September 2011, 14:30:23 schrieb address@hidden:
> Passes make and reg tests
>
> http://codereview.appspot.com/5032047/
It would be really nice to have a better summary than "Fix 155". This does not
tell me ANYTHING about what this is all about, so I'll have to click on the
link to find out what this patch is for (and either remember this, or have to
visit the rietveld page again when I can't remember what all the lilypond bug
numbers are about).
Or of course I simply ignore that mail, which is usually what I do.
So, please add a short summary to the patch summary! This is also preferred
for the actual git commit, since having a patch summary of "Fix 155" is just
as un-helpful in the the git history as it is here.
Cheers,
Reinhold
--
------------------------------------------------------------------
Reinhold Kainhofer, address@hidden, http://reinhold.kainhofer.com/
* Financial & Actuarial Math., Vienna Univ. of Technology, Austria
* http://www.fam.tuwien.ac.at/, DVR: 0005886
* LilyPond, Music typesetting, http://www.lilypond.org