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

Ramiro Polla ramiro.polla at gmail.com
Wed Oct 24 21:25:18 CEST 2012

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.

More information about the ffmpeg-devel mailing list