[FFmpeg-devel] dshow: patches for adding some debug output, using more standardized codec table for lookup

[Ramiro Polla]

On Mon, Oct 15, 2012 at 2:40 PM, Roger Pack <rogerdpack2 at gmail.com> wrote:
> On Mon, Oct 1, 2012 at 2:38 PM, Ramiro Polla <ramiro.polla at gmail.com> wrote:
>> On Mon, Oct 1, 2012 at 4:41 PM, Roger Pack <rogerdpack2 at gmail.com> wrote:
>>>> Patch 0001 is ok. Patch 0002 seems to miss MKTAG('m', 'j', 'p', 'g').
>>>
>>> Does anybody use lower case mjpg in the wild...it seemed from the
>>> devices I tested that nobody does but I'd be happy to be corrected...
>>
>> I don't know. The tags were added in commit svn-22004:
>> "vfwcap: support MJPG compressed streams.
>>
>> Patch by Nash Tsai <nash dot tsai at gmail dot com>"
>>
>> Perhaps you could ask him. Otherwise either add the tag to riff.c or
>> drop the tag and wait for someone to complain =)
>
> Ok Ramiro, no response from him, after doing some google search it
> seems that MJPG is common, but I can find basically no instances of
> mjpg "in the wild." Do you think the patch is ok as written?

Yes. Thank you for working on dshow and I'm sorry for the delay.