[MPlayer-cvslog] r28468 - trunk/libvo/vo_direct3d.c
Diego Biurrun
diego at biurrun.de
Thu Feb 5 14:05:45 CET 2009
On Thu, Feb 05, 2009 at 10:06:10AM +0100, gogothebee wrote:
>
> Log:
> This patch unifies info/error messages
> to a common style: <vo_direct3d> in the beginning of each message and "." at the end. Messages are shorter/more descriptive.
Congrats for your first commit from me as well.
Now a few words about your commit message:
- Please keep lines below 80 characters where reasonable.
- A patch is not the same thing as a commit :)
- Do not indirectly describe what your commit does and talk about your
commit as an entity in your commit message. Just say it directly.
I'm not saying "This email is supposed to teach you to write better
commit messages." because this is an email and it is obvious that you
know this is an email. So instead of "This patch/commit unifies.."
you cut out the middlemen and say "Unify...".
- DOCS/tech/svn-howto.txt has a short paragraph about changing commit
messages.
Diego
More information about the MPlayer-cvslog
mailing list