[FFmpeg-devel] [PATCH] socklen_t definition

[Dave Yeo]

Reimar Doeffinger wrote:
> Hello,
> On Mon, Sep 17, 2007 at 12:35:46PM +0200, Diego Biurrun wrote:
>> Even more important: Reimar came up with a header file that provides the
>> correct definition.  So why can't this be used instead of adding this
>> (possibly brittle as explained by Mans) check?
> 
> I'm not sure that is a proper header file, at least it's not a system
> one...
> But it seems that _socklen_t is in some header file, maybe using that is
> good enough?

Grepping include I found in <386/ansi.h>
/*
  * Types which are fundamental to the implementation and must be declared
  * in more than one standard header are defined here.  Standard headers
  * then use:
  *      #ifdef  _BSD_SIZE_T_
  *      typedef _BSD_SIZE_T_    size_t;
  *      #undef  _BSD_SIZE_T_
  *      #endif
  */
...
#define _BSD_SOCKLEN_T_ __uint32_t              /* socklen_t (duh) */
...

And in <sys/_type.h>
...
typedef __uint32_t      __socklen_t;
...
So it seems that it should be uint32_t. Also __uint32_t is typedef as
unsigned int in <386/_types.h>.
Dave