[MPlayer-dev-eng] [PATCH] 'socklen_t' definition

Diego Biurrun diego at biurrun.de
Fri Aug 1 07:34:59 CEST 2008


On Thu, Jul 31, 2008 at 11:01:21PM +0900, KO Myung-Hun wrote:
> 
> Diego Biurrun wrote:
> > On Thu, Jul 31, 2008 at 12:30:17AM +0900, KO Myung-Hun wrote:
> >   
> >> Diego Biurrun wrote:
> >>     
> >>> On Wed, Jul 30, 2008 at 12:06:36AM +0900, KO Myung-Hun wrote:
> >>>       
> >>>> Some system such as OS/2, 'socklen_t is not defined. So we need to check
> >>>> its definition and if needed, should defined it.
> >>>>
> >>>> But, this check and definition can conflict with other systems. So set
> >>>> its default value to 'HAVE', and check only on OS/2, finally if needed
> >>>> define 'socklen_t'.
> >>>>
> >>>> Can this approach make a trouble with systems other than OS/2 ?
> >>>>         
> >>> I just committed a different check.  Testing on Windows is welcome.
> >>>       
> >> Additional patch.
> >>
> >> 1. _socklen_t should be initialized to 'no'
> >
> > I don't see what this helps.
> 
> At least, it display result is 'no' instead of emptiness when 
> 'socklen_t' is not found.

True, changed.

Diego



More information about the MPlayer-dev-eng mailing list