Michael Niedermayer <[email protected]> added the comment: On Mon, Oct 12, 2009 at 11:33:40PM +0000, Baptiste Coudurier wrote: > On 10/12/2009 12:30 PM, Michael Niedermayer wrote: > > On Sun, Oct 11, 2009 at 11:21:29PM +0000, Carl Eugen Hoyos wrote: > >> > >> Following inlined patch does not work (does not change output) although > >> extradata is set: > > > > maybe something like the following pseudo patch would work? > > > > - if(s->avctx->codec->id==CODEC_ID_AMV) { > > + if(s->flipped) { > > //picture should be flipped upside-down for this codec > > assert(!(s->avctx->flags& CODEC_FLAG_EMU_EDGE)); > > data[c] += (linesize[c] * (s->v_scount[i] * (8 * s->mb_height > > -((s->height/s->v_max)&7)) - 1 )); > > linesize[c] *= -1; > > } > > ... > > +if(some header indicates intels jpeg stuff || > > s->avctx->codec->id==CODEC_ID_AMV) > > + s->flipped=1; > > Isn't codec_tag sufficient ?
for avi,mov,nut,... yes but someone could store these raw or as individual images like with -f image2, for these cases not depending on codec_tag would be better. [...] _____________________________________________________ FFmpeg issue tracker <[email protected]> <https://roundup.ffmpeg.org/roundup/ffmpeg/issue1464> _____________________________________________________
