Justin Ruggles <[email protected]> writes:

> On 12/30/2011 02:26 PM, Justin Ruggles wrote:
>
>> On 12/30/2011 02:08 PM, Måns Rullgård wrote:
>> 
>>> Derek Buitenhuis <[email protected]> writes:
>>>
>>>> From: Michael Niedermayer <[email protected]>
>>>>
>>>> Signed-off-by: Michael Niedermayer <[email protected]>
>>>> ---
>>>>  libavformat/rawdec.c |    2 +-
>>>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>>>
>>>> diff --git a/libavformat/rawdec.c b/libavformat/rawdec.c
>>>> index 3e2dd05..33a0447 100644
>>>> --- a/libavformat/rawdec.c
>>>> +++ b/libavformat/rawdec.c
>>>> @@ -201,7 +201,7 @@ AVInputFormat ff_latm_demuxer = {
>>>>  #endif
>>>>
>>>>  #if CONFIG_MJPEG_DEMUXER
>>>> -FF_DEF_RAWVIDEO_DEMUXER(mjpeg, "raw MJPEG video", NULL, "mjpg,mjpeg", 
>>>> CODEC_ID_MJPEG)
>>>> +FF_DEF_RAWVIDEO_DEMUXER(mjpeg, "raw MJPEG video", NULL, "mjpg,mjpeg,mpo", 
>>>> CODEC_ID_MJPEG)
>>>>  #endif
>>>
>>> Is this a common filename suffix?  Why is it not detected by probing?  I
>>> don't like the smell of this patch.
>> 
>> 
>> I found some samples, and the format is not detected by probing. I'm not
>> sure why though.
>> 
>> http://www.steves-digicams.com/camera-reviews/fujifilm/finepix-real-3d-w3/sample-photos-122.html
>
> Also found a spec if someone wants to try to support probing.
>
> http://www.cipa.jp/english/hyoujunka/kikaku/pdf/DC-007_E.pdf

Looks like the format merits a full demuxer.  Treating it as plain MJPEG
is certainly wrong.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to