[MPlayer-dev-eng] [PATCH] tremor: some workaround for something

Arpi arpi at thot.banki.hu
Tue Jan 28 20:40:49 CET 2003


Hi,

note that i don't know vorbis API at all, but i assumed that:

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

vorbis_synthesis_pcmout() always return the number of PCM samples available
in codec's buffer, and it returns <=0 if no samples available yet

and that

> +	    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);

vorbis_synthesis+vorbis_synthesis_blockin always processes op.bytes bytes
from op.packet.

if these assumptions are wrong, then forget my patch, it won't work.


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