[FFmpeg-devel] [PATCH]Recognize

Michael Niedermayer michaelni at gmx.at
Fri Nov 2 20:33:38 CET 2012


On Fri, Nov 02, 2012 at 07:21:14PM +0000, Carl Eugen Hoyos wrote:
> Michael Niedermayer <michaelni <at> gmx.at> writes:
> 
> > > >              //Read QT version 1 fields. In version 0 these do not exist.
> > > >              av_dlog(c->fc, "version =%d, isom =%d\n",version,c->isom);
> > > > -            if (!c->isom) {
> > > > +            if (!c->isom ||
> > > > +                strstr(av_dict_get(c->fc->metadata, "compatible_brands", 
> NULL,  AV_DICT_MATCH_CASE)->value,
> > "qt  ")) {
> > > >                  if (version==1) {
> > > >                      sc->samples_per_frame = avio_rb32(pb);
> > > >                      avio_rb32(pb); /* bytes per packet */
> > > 
> > > Might be ok.
> > 
> > Is this special case specific to this use of isom or could isom
> > itself be changed based on the existence of qt in compatible_brands
> > ?
> 
> You mean like my original patch (with strstr)?

yes

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121102/2c7f0fef/attachment.asc>


More information about the ffmpeg-devel mailing list