On Tue, May 28, 2013 at 12:50:03PM +0300, Martin Storsjö wrote:
> On Tue, 28 May 2013, Kostya Shishkov wrote:
> 
> >On Tue, May 28, 2013 at 11:26:04AM +0300, Martin Storsjö wrote:
> >>From: Michael Niedermayer <[email protected]>
> >
> >I'd say that commit message can be improved and the motion compensation part
> >needs to be reindented, otherwise looks OK.
> 
> Reindented the block that was left unindented, what would be a
> better commit message here?
> 
> "Add support for averaging and for specifying the direction to
> vc1_mc_4mv_chroma4()"?

Handle top and bottom blocks in vc1_mc_4mv_chroma4() differently if needed.

And in expanded log message it would be good to mention what was changed
exactly, e.g.:

Now it can use different references for those blocks and even use averaging.

(I'm not good at composing commit messages myself).
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to