fix missing check against lame_init_params that was leading to video only files on low (under 32) audio bitrates

git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@14763 b3059339-0415-0410-9bf9-f77b7e298cf2
This commit is contained in:
reynaldo 2005-02-22 15:13:28 +00:00
parent bd10844282
commit 451ec22f8f
3 changed files with 10 additions and 1 deletions

View File

@ -337,6 +337,9 @@ static char help_text[]=
"mw-us => 40kbps/mono voice => 56kbps/mono\n"\
"fm/radio/tape => 112kbps hifi => 160kbps\n"\
"cd => 192kbps studio => 256kbps"
#define MSGTR_LameCantInit "can't set Lame options, check bitrate/samplerate,"\
" some very low bitrates (<32) need lower samplerates (i.e. -srate 8000). "\
"if everything else fails, try a preset."
#define MSGTR_ConfigfileError "configfile error"
#define MSGTR_ErrorParsingCommandLine "error parsing cmdline"
#define MSGTR_VideoStreamRequired "Video stream is mandatory!\n"

View File

@ -308,6 +308,9 @@ static char help_text[]=
"mw-us => 40kbps/mono voice => 56kbps/mono\n"\
"fm/radio/tape => 112kbps hifi => 160kbps\n"\
"cd => 192kbps studio => 256kbps"
#define MSGTR_LameCantInit "No se pudo setear las opciones de Lame, revise el"\
" bitrate/samplerate, algunos bitrates muy bajos (<32) necesitan una tasa de"\
" muestreo más baja (ej. -srate 8000). Si todo falla, pruebe con un preset."
#define MSGTR_ConfigfileError "error en archivo de configuración"
#define MSGTR_ErrorParsingCommandLine "error en parametros de la línea de comando"
#define MSGTR_VideoStreamRequired "¡El flujo de video es obligatorio!\n"

View File

@ -1110,7 +1110,10 @@ if(lame_param_preset != NULL){
lame_presets_set(lame,lame_param_fast, (lame_param_vbr==0), lame_param_preset);
}
#endif
lame_init_params(lame);
if(lame_init_params(lame) == -1){
mp_msg(MSGT_MENCODER, MSGL_FATAL, MSGTR_LameCantInit);
mencoder_exit(1,NULL);
}
if(verbose>0){
lame_print_config(lame);
lame_print_internals(lame);