[MPlayer-dev-eng] [PATCH] Fix packed YUV in dshow vo

Laurent laurent.aml at gmail.com
Mon Oct 6 16:30:36 CEST 2008


On 10/5/08, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
> On Sun, Oct 05, 2008 at 11:04:22AM +0200, Sascha Sommer wrote:
> > Not sure if MPlayer defines a MIN macro. If not you might introduce one in
> > vo_directx.c. You can then simply use
> >
> >                       minStride = MIN(srcStride,dstStride);
> >
> > instead of the if else block above.
>
> Please use FFMIN from libavutil (you might have to include
> libavutil/common.h).
> Maybe that header is a bit overkill but it is better than
> duplicated/obfuscated code IMO.
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>

Here is a simplified version of the patch:
* ugly decls fixed
* use FFMIN
* debug msg removed.

Thanks,
Laurent
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: libvo_directx.c.patch
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20081006/ed28d1b3/attachment.asc>


More information about the MPlayer-dev-eng mailing list