[Ffmpeg-devel] More of the BeOS fixes...

Reimar Döffinger Reimar.Doeffinger
Thu Mar 8 21:17:14 CET 2007


Hello,
On Thu, Mar 08, 2007 at 08:26:52PM +0100, Fran?ois Revol wrote:
> The recent removal of PRI* and *INT64_C broke the BeOS build, and as
> some of you probably noticed, I got quite pissed off at it after
> cleaning the beos port for 2 weeks or so...

Well, the comment for Windows was to provide your own proper stdint.h/inttypes.h
headers.
I have no real authority here, but:

> Index: libavformat/network.h
> ===================================================================
> --- libavformat/network.h	(revision 8296)
> +++ libavformat/network.h	(working copy)
> @@ -24,7 +24,22 @@
>  #include <sys/types.h>
>  #include <sys/socket.h>
>  #include <netinet/in.h>
> +#ifdef HAVE_ARPA_INET_H
>  #include <arpa/inet.h>
> +#else
> +#    if !defined(HAVE_INET_ATON)
> +/* in os_support.c */
> +int inet_aton (const char * str, struct in_addr * add);
> +#    endif
> +#endif
>  #include <netdb.h>
>  
> +#if !defined(IP_ADD_MEMBERSHIP)
> +/* just so structs have a storage size */
> +struct ip_mreq {
> +    struct in_addr imr_multiaddr;  /* IP multicast address of group */
> +    struct in_addr imr_interface;  /* local IP address of interface */
> +};
>  #endif
> +
> +#endif

Wouldn't it make more sense to put these into os_support.h if the
function itself is already in os_support.c?

Greetings,
Reimar D?ffinger




More information about the ffmpeg-devel mailing list