[FFmpeg-devel] [PATCH] Add the 'broadcast' option to udp protocol

[Michael Niedermayer]

On Fri, Aug 10, 2012 at 01:05:20PM +0200, Federico Fuga wrote:
> This option specify that the provided address is a net broadcast address.
> This is needed by some platform to transmit to and receive from network broadcast addresses.
> Note that different, although Unix-based, Operating Systems behaves
> differently with network broadcast addresses, certains allows transmissions,
> other allows receiving only, other (like android) none of them. When in doubt,
> specify this option when using broadcast addresses.
> ---
>  doc/protocols.texi |    8 ++++++++
>  libavformat/udp.c  |   15 ++++++++++++++-
>  2 files changed, 22 insertions(+), 1 deletion(-)

patch does not apply cleanly

[...]
>          if (setsockopt (udp_fd, SOL_SOCKET, SO_REUSEADDR, &(s->reuse_socket), sizeof(s->reuse_socket)) != 0)
>              goto fail;
>      }
> -
> +    
> +    if (s->is_broadcast) {
> +        if (setsockopt(udp_fd, SOL_SOCKET, SO_BROADCAST, &(s->is_broadcast), sizeof(s->is_broadcast)) != 0) 
> +            goto fail;
> +    }
> +    

trailing whitespace is forbidden in ffmpeg git

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Observe your enemies, for they first find out your faults. -- Antisthenes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120816/1f472628/attachment.asc>