[MPlayer-dev-eng] missing check on mencoder leads to video only files without warning/error

[Reynaldo H. Verdejo Pinochet]

On Mon, Feb 21, 2005 at 03:44:42PM -0300, Reynaldo H. Verdejo Pinochet wrote:
> a check should be done against lame_init_params to ensure proper audio
> encoding, we cant just *trust* options to be ok, try with -oac mp3lame 
> -lameopts cbr:br=16 for an example.
> 
> The imcluded patch does the needed check and makes mencoder bail
> with an error when lame_init_params returns -1 (error, see lame.h).
> 
> will wait for comments till midnight.
> 
> Reynaldo
> 

commited.

   Reynaldo
-------------- 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/mplayer-dev-eng/attachments/20050222/e189693d/attachment.pgp>