[FFmpeg-devel] [PATCH] fix stream copy sample_aspect_ratio

Michael Niedermayer michaelni
Sat Sep 27 12:16:05 CEST 2008


On Sat, Sep 27, 2008 at 09:33:38AM -0000, Wolfram Gloger wrote:
> Hi,
> 
> > i meant
> > 
> > + video_enc->sample_aspect_ratio =3D
> >   st->sample_aspect_ratio =3D av_d2q(frame_aspect_ratio*frame_height/frame_=
> > width, 255);
> 
> Whatever...
> 
> Regards,
> Wolfram.
> 
> --- trunk/ffmpeg.c	Fri Sep 26 09:33:03 2008
> +++ ffmpeg-wg/ffmpeg.c	Sat Sep 27 11:30:08 2008
> @@ -3025,7 +3025,8 @@
>      if (video_stream_copy) {
>          st->stream_copy = 1;
>          video_enc->codec_type = CODEC_TYPE_VIDEO;
> -        st->sample_aspect_ratio = av_d2q(frame_aspect_ratio*frame_height/frame_width, 255);
> +        video_enc->sample_aspect_ratio =
> +            st->sample_aspect_ratio = av_d2q(frame_aspect_ratio*frame_height/frame_width, 255);
>      } else {

no this mixes a cosmetic change with a fuctional one, what i meant was
adding one line is enough, there is no need to change the existing line.

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored awnser#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080927/5ca643c6/attachment.pgp>



More information about the ffmpeg-devel mailing list