[MPlayer-dev-eng] [PATCH] tremor: v2.0 some workaround for

Arpi arpi at thot.banki.hu
Tue Jan 28 23:56:23 CET 2003


Hi,

> So
> 
> > ehh
> >
> > -         while((samples=vorbis_synthesis_pcmout(&ov->vd,&pcm))<=0){
> > -           op.bytes = ds_get_packet(sh->ds,&op.packet);
> > -           if(op.bytes<=0) break;
> > -           if(vorbis_synthesis(&ov->vb,&op)==0) /* test for success! */
> > -             vorbis_synthesis_blockin(&ov->vd,&ov->vb);
> 
> Here should come first something like 
> +	if (vorbis_synthesis_pcmout(&ov->vd,&pcm)>0) break;  
> 
> ?

why?

while((samples=vorbis_synthesis_pcmout(&ov->vd,&pcm))<=0){
}

just do the same


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu
    "However, many people beg for its inclusion in Debian. Why?" - Gabucino
  "Because having new software in Debian is good." - Josselin Mouette
"Because having good software in Debian is new." - Gabucino


More information about the MPlayer-dev-eng mailing list