Signed-off-by: NMans Rullgard <mans@mansr.com>

The only symbol this file uses from unistd.h is isatty().  By
including the header only when this function is used, the file
can be built on systems without unistd.h (which presumably also
lack isatty).
Signed-off-by: NMans Rullgard <mans@mansr.com>

This avoids some warnings about redundant declarations.
Signed-off-by: NMans Rullgard <mans@mansr.com>

Add -D_XOPEN_SOURCE=600 to host cflags to make the constants in math.h
available. Include math.h where necessary and remove redundant M_PI
defines.

Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMans Rullgard <mans@mansr.com>

It's redundant, since the input buffer is passed as a parameter to the
filter_samples() callback, and can lead to stale pointers remaining on
the link.

It will be used for audio too.

Its documentation states that it is allocated/freed by the caller, but
it is declared as an AV_OPT_TYPE_STRING AVOption. Since
36773283 the AVOptions system frees
strings automatically. This can be considered an API break, since it
won't work when the caller doesn't use av_malloc() to allocate the
memory or wants to use the string after closing the codec.

Since there is not much value in this field being an AVOption, the best
solution is to remove it from the options table.

'channel_layout' not 'channels' from the stored configuration should go
to AVCodecContext's 'channel_layout'.

Based on patch by Ronald S. Bultje.
Signed-off-by: NMans Rullgard <mans@mansr.com>

Signed-off-by: NMans Rullgard <mans@mansr.com>

Signed-off-by: NMans Rullgard <mans@mansr.com>

While these defines are not defined by the C standard they are
standardized as X/Open System Interfaces Extension. We use the
appropiate _XOPEN_SOURCE define to make them available. They
seem to be available on all FATE configs since the constants
are used in files where mathematics.h is not included.

Signed-off-by: NMans Rullgard <mans@mansr.com>

These files do not use anything provided by unistd.h.
Signed-off-by: NMans Rullgard <mans@mansr.com>

Signed-off-by: NMans Rullgard <mans@mansr.com>

Signed-off-by: NMans Rullgard <mans@mansr.com>

If stdin has been closed, 0 is a valid socket descriptor.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Make sure we actually have an error code in ret, in case
getsockopt failed.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Also use ff_neterrno() instead of errno directly (which doesn't work
on windows), for getting the error code.
Signed-off-by: NMartin Storsjö <martin@martin.st>

getnameinfo doesn't set errno on failure, it returns an error code,
which should be handled by gai_strerror instead of the normal
strerror.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

This avoids warnings.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Rtmpt is effectively half duplex - the server can't return any
data unless we send a request (to which the server responds). If
we don't have any data to send currently, and the server didn't
return any data either, wait a little before doing the next request.

This avoids busy looping with idle posts with empty replies, while
waiting for more data from the server.
Signed-off-by: NMartin Storsjö <martin@martin.st>

The check uses check_func_header, since this function is
conditionally available depending on the targeted MSVCRT
version.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Add a note that pkt->data and pkt->size must be initialized.
Signed-off-by: NMartin Storsjö <martin@martin.st>