Please do not top-post.

On Fri, Jan 06, 2012 at 07:26:41PM +0200, Laurentiu Cristian Ion wrote:
> 2012/1/6 Reinhard Tartler <[email protected]>
> 
> > On Fr, Jan 06, 2012 at 16:54:40 (CET), Laurentiu Ion wrote:
> >
> > > ---
> > >  libavcodec/flicvideo.c |  161
> > +++++++++++++++++++++++-------------------------
> > >  1 files changed, 76 insertions(+), 85 deletions(-)
> > >
> > > diff --git a/libavcodec/flicvideo.c b/libavcodec/flicvideo.c
> > > index 8e8a813..d96375a 100644
> > > --- a/libavcodec/flicvideo.c
> > > +++ b/libavcodec/flicvideo.c
> > > @@ -41,6 +41,8 @@
> >
> > [...]
> >
> > poor commit message. please try to be more descriptive what this change
> > does and how a potential reviewer is supposed to verify it.
> 
> well, before I was told the commit message was too long...

The problem was not the length of the log message, but that it contained
the same sentence four times.  Replace your log message with

  flicvideo: <short one-line description of your change>
  <empty line>
  <more detailed description of your change, if one line is not enough>

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to