Makes passing to configure
--toolchain=gcc-asan --cross-prefix=armv7a-hardfloat-linux-gnueabi-
work as intended.

CC: libav-stable@libav.org
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Incidentally `-y` also collides with avconv global options.

Update x11grab to match and document the option.

CC: libav-stable@libav.org
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Many of these functions were named foo_free_context, and since
the functions no longer should free the context itself, only
allocated elements within it, the previous naming was slightly
misleading.
Signed-off-by: NMartin Storsjö <martin@martin.st>

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

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

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

This is different from how it is handled in codecs/demuxers/muxers
though (where the close function isn't called if the open function
failed), but since the number of depacketizers that have an .init
function is quite limited, this is easy to change.

The main point is that if the init function failed, we shouldn't
try to use that depacketizer at all - this makes sure that the
parse function doesn't need to check for the things that were
initialized in the init function.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This makes it more consistent with depacketizers that don't have any
.free function at all, where the payload context is freed by the
surrounding framework. Always free the context in the surrounding
framework, having the individual depacketizers only free any data
they've specifically allocated themselves.

This is similar to how this works for demuxer/muxers/codecs - a
component shouldn't free the priv_data that the framework has
allocated for it.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Always use the .priv_data_size field instead.
Signed-off-by: NMartin Storsjö <martin@martin.st>

The ugly error handling in rdt gets improved in a later commit.
Signed-off-by: NMartin Storsjö <martin@martin.st>

These functions are far from performance critical, so there's no
point in marking them as inline.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This makes it clear that the individual parsing functions can't
touch the parsed out value.
Signed-off-by: NMartin Storsjö <martin@martin.st>

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

The free function of a depacketizer won't be called if data is NULL.
Signed-off-by: NMartin Storsjö <martin@martin.st>

They share a great deal of common structure; only a few minor
bits in the headers differ.

This also fixes an off-by-one in sending of the last fragment
of large HEVC nals (where it previously sent len+2 bytes, even
if it should have been len+RTP_HEVC_HEADERS_SIZE aka len+3).
Signed-off-by: NMartin Storsjö <martin@martin.st>

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

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

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

This is the common style for such comments.
Signed-off-by: NMartin Storsjö <martin@martin.st>

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

This can reduce the amount of boilerplate in simple depacketizers.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This avoids implementing a full function just to set this one
field.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This allows getting rid of quite a bit of boilerplate in depacketizers.

The default value (initializing need_parsing to 0, aka
AVSTREAM_PARSE_NONE) is the same as it is initialized to by default
in AVStream.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This makes it match the other depacketizers.
Signed-off-by: NMartin Storsjö <martin@martin.st>

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

This avoids allocating space for a too large buffer for all the
name strings.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Nothing in this file use any network functions.
Signed-off-by: NMartin Storsjö <martin@martin.st>

There's no point in adding padding in the allocation of a depacketizer
specific context struct.
Signed-off-by: NMartin Storsjö <martin@martin.st>

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

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

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

Map this to AV_CODEC_ID_TEXT.
Signed-off-by: NMartin Storsjö <martin@martin.st>

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

The codec is referred to as DCA in other parts of libav.

When a client behind a NAT issues a pause command, and stay paused for a
long time, the router may stop the RTP/RTCP port redirection. Resend the
hole punching packets before each PLAY command to cause the router to
restart the port redirection in that case.

Move the existing code for sending the packets from the SETUP phase
to the PLAY phase.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Bug-Id: CID 1257500
CC: libav-stable@libav.org
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Bug-Id: CID 1257501
CC: libav-stable@libav.org
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>