[Ffmpeg-devel] Third Try: Set bit_rate for asf format
Michael Niedermayer
michaelni
Sat Apr 28 10:35:32 CEST 2007
Hi
On Sat, Apr 28, 2007 at 10:49:47AM +0800, Zuxy Meng wrote:
> Hi,
>
> 2007/4/17, Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> > Hello,
> > On Tue, Apr 17, 2007 at 07:59:24PM +0800, Zuxy Meng wrote:
> > > @@ -504,11 +509,14 @@
> > >
> > > for(i=0; i<128; i++){
> > > int stream_num= asf->asfid2avid[i];
> > > - if(stream_num>=0 && dar[i].num>0 && dar[i].den>0){
> > > + if(stream_num>=0){
> > > AVCodecContext *codec= s->streams[stream_num]->codec;
> > > - av_reduce(&codec->sample_aspect_ratio.num,
> > > - &codec->sample_aspect_ratio.den,
> > > - dar[i].num, dar[i].den, INT_MAX);
> > > + if (!codec->bit_rate)
> > > + codec->bit_rate = bitrate[i];
> > > + if (dar[i].num > 0 && dar[i].den > 0)
> > > + av_reduce(&codec->sample_aspect_ratio.num,
> > > + &codec->sample_aspect_ratio.den,
> > > + dar[i].num, dar[i].den, INT_MAX);
> >
> > Hmm... you only change the indentation for that av_reduce? That should
> > be done in a separate step I think.
>
> No other comments apart from this? I'm still waiting for Michael's....
i agree with reimar that the indention change should be seperate
except that iam fine with the patch
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070428/9d0648af/attachment.pgp>
More information about the ffmpeg-devel
mailing list