[FFmpeg-devel] [PATCH] segment: don't copy codec_tag when not available
Michael Niedermayer
michaelni at gmx.at
Tue Nov 20 19:17:48 CET 2012
On Sat, Nov 17, 2012 at 06:17:59PM +0100, Stefano Sabatini wrote:
> On date Friday 2012-11-16 15:55:50 +0100, Michael Niedermayer encoded:
> > On Fri, Nov 16, 2012 at 03:11:50AM +0900, ChanMin Kim wrote:
> > > It looks like gmail added unnecessary newline in the patch.
> > >
> > > + av_codec_get_id (oc->oformat->codec_tag,
> > > icodec->codec_tag) == codec->codec_id ||
> > >
> > > should be in one line.
> > >
> > > + av_codec_get_id (oc->oformat->codec_tag,
> > > icodec->codec_tag) == codec->codec_id ||
> >
> > please resubmit the patch as an attachment so its not mangled
>
> Updated.
applied
> I'm not yet sure about the logic of the patch.
if you find a problem tell me, ill fix it ASAP, but to me to looks ok
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
DNS cache poisoning attacks, popular search engine, Google internet authority
dont be evil, please
-------------- 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/20121120/46bb3324/attachment.asc>
More information about the ffmpeg-devel
mailing list