[FFmpeg-cvslog] r14363 - in trunk/libavcodec: ra288.c ra288.h

Uoti Urpala uoti.urpala
Thu Jul 24 19:16:31 CEST 2008


On Thu, 2008-07-24 at 18:24 +0200, Michael Niedermayer wrote:
> and reading svnlog with a (non modified) MUA will also show the filename

It will be visible, but not as easily readable. You could also say about
a commit "it's clearly visible from the patch that this removes support
for mpeg2, no need to mention that part in the commit message". Reading
the filename list is usually less extra effort than reading the patch,
but adding that information in the log message is also usually less
effort than describing the patch contents.


I think the log mails could be significantly improved. The current
subject lines make it hard to find the correct thread unless you
remember the revision number. Using the first line of the commit message
as the subject instead would be much more useful. And then having proper
context information in the message is important again - even if you'd
also add the filename list in the subject both the log message and
filename list would not be visible for many commits unless you'd
configure your MUA to use a lot of screen space per subject line.

So instead of of
[FFmpeg-cvslog] r14265 - trunk/libavutil/intreadwrite.h
[FFmpeg-cvslog] r14275 - trunk/libavutil/intreadwrite.h
you'd have
[FFmpeg-cvslog] r14265 - Rearrange AV_[RW][BL]*() macros
[FFmpeg-cvslog] r14275 - intreadwrite: support DEC compiler __unaligned type qualifier






More information about the ffmpeg-cvslog mailing list