[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