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

[Roger Pack]

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?
-r