On Fri, Sep 14, 2012 at 8:31 AM, Dan Dennedy <[email protected]> wrote:
> On Fri, Sep 14, 2012 at 2:29 AM, j-b-m <[email protected]> wrote:
>> On Friday 14 September 2012 10:24:37 j-b-m wrote:
>>> On Thursday 13 September 2012 22:44:57 GitHub wrote:
>>> > https://github.com/mltframework/mlt/commit/ac034ef73a340fe3402968a7b3a3f69
>>> > d
>>> > e4a3c53a Author: Dan Dennedy <[email protected]>
>>> >
>>> >   Date:   2012-09-13 (Thu, 13 Sep 2012)
>>> >
>>> >   Changed paths:
>>> >     M src/modules/sdl/consumer_sdl.c
>>> >
>>> >   Log Message:
>>> >   -----------
>>> >   change sdl consumer to use yuv420p
>>> >
>>> > based on a patch by Steinar Gunderson
>>>
>>> This commit breaks the color producer:
>>>
>>> melt color:red
>
> OK, I will look into it.
>
>>>
>>> Shows only artifact lines and crashes on exit. I guess it is because the
>>> color producer code in "producer_get_image" does not handle yuv420p and
>>> defaults to rgba.
>>
>> I managed to fix the color producer by adding a routine to handle yuv420p
>
> That should not be necessary. Some conversion is not happening. I
> found that adding "-consumer sdl mlt_image_format=yuv422" resolves it.
>
>> format, but unfortunately scaling is also broken by this yuv420p change:

It turns out that the color producer was not prepared to handle a
request for yuv420p. Therefore, this change
(mlt_consumer.mlt_image_format=yuv420p) requires a review and test of
all of the services. Since this is not a quick fix I reverted the
patch for now.

>>
>> melt -profile dv_pal_wide hdv.mp4
>>
>> produces a corrupted image.
>>
>> regards
>> jb
>>
>>
>>
> --
> +-DRD-+



-- 
+-DRD-+

------------------------------------------------------------------------------
Got visibility?
Most devs has no idea what their production app looks like.
Find out how fast your code is with AppDynamics Lite.
http://ad.doubleclick.net/clk;262219671;13503038;y?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html
_______________________________________________
Mlt-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mlt-devel

Reply via email to