Mercurial > mplayer.hg
view subopt-helper.h @ 24590:2c238fa777ff
ao_alsa: Fix get_space() return values larger than buffersize
After a buffer underrun the ALSA get_space() function sometimes returned
values larger than the ao had set in ao_data.buffersize. Fix this by
replacing the old check against MAX_OUTBURST by one against
ao_data.buffersize. There should be no need for the MAX_OUTBURST check;
the current MPlayer side should no longer have any constant limit on the
amount of data an ao can buffer or request at once.
The get_space() values larger than ao_data.buffersize triggered errors
in audio decoding causing the current attempt to fill audio buffers to
be aborted. I'm not sure how often that caused behavior noticeably worse
then an underrun already is.
author | uau |
---|---|
date | Mon, 24 Sep 2007 21:49:58 +0000 |
parents | 5949a654e2d4 |
children | 3baf6a2283da |
line wrap: on
line source
#ifndef SUBOPT_HELPER_H #define SUBOPT_HELPER_H /** * \file subopt-helper.h * * \brief Datatype and functions declarations for usage * of the suboption parser. * */ #define OPT_ARG_BOOL 0 #define OPT_ARG_INT 1 #define OPT_ARG_STR 2 #define OPT_ARG_MSTRZ 3 ///< A malloced, zero terminated string, use free()! #define OPT_ARG_FLOAT 4 typedef int (*opt_test_f)(void *); /** simple structure for defining the option name, type and storage location */ typedef struct opt_s { const char * name; ///< string that identifies the option int type; ///< option type as defined in subopt-helper.h void * valp; ///< pointer to the mem where the value should be stored opt_test_f test; ///< argument test func ( optional ) int set; ///< Is set internally by the parser if the option was found. ///< Don't use it at initialization of your opts, it will be ///< overriden anyway! } opt_t; /** parses the string for the options specified in opt */ int subopt_parse( char const * const str, opt_t * opts ); /*------------------ arg specific types and declaration -------------------*/ typedef struct strarg_s { int len; ///< length of the string determined by the parser char const * str; ///< pointer to position inside the parse string } strarg_t; int int_non_neg( int * i ); int int_pos( int * i ); int strargcmp(strarg_t *arg, const char *str); int strargcasecmp(strarg_t *arg, char *str); #endif