[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