[Mplayer-cvslog] CVS: main cfg-common.h,1.62,1.63 cfg-mplayer.h,1.170,1.171 mencoder.c,1.173,1.174 mplayer.c,1.587,1.588

[Anders Johansson]

Hi,

> Hi,
> 
> > Index: mencoder.c
> > ===================================================================
> 
> > @@ -671,7 +672,7 @@
> >      } else {
> >  	mux_a->wf = malloc(sizeof(WAVEFORMATEX));
> >  	mux_a->wf->nBlockAlign = 1; //mux_a->h.dwSampleSize;
> > -	mux_a->wf->wFormatTag = sh_audio->format;
> > +	mux_a->wf->wFormatTag = audio_output_format?audio_output_fo
> > rmat:sh_audio->format;
> 
> 
> this is nonsense.
> you cannot override format for audio copy mode...
> you may do it, after checking that format is PCM, and initializing filter
> layer, and calling teh filter layer. but there is -oac pcm for this purpose.

Just me being stupid, thanks for fixing. I had to leave straight after
committing, so I didn't see this post until this morning otherwise I
would have reversed myself.

> btw, why is -af mplayer-only?

You said that mencoder can have multiple streams. Suppose there needs
to be separate cmdline switches for each of those streams. Some more
thinking needs to be done here before any code is written. Also I am
not experienced with the code in mencoder, it is not so interesting to
me, so if anyone else feels like taking care of this I would be happy.


> 
> A'rpi / Astral & ESP-team
> 

//Anders