Mainly clean up the RTP statistics code, plus a few other obviously
misindentend lines.

Remove some useless comments, de-doxygenize some comments,
add spacing around operators and fix a typo.
Signed-off-by: Martin Storsjö <martin@martin.st>

The muxer already rejects unsupported pixel formats, reject also
unsupported codecs to prevent dangerous misuses.

This changes the event_cmp() function to the correct signature,
avoiding an ugly cast.
Signed-off-by: Mans Rullgard <mans@mansr.com>

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

Indexing across array boundaries is not allowed by C99.
Signed-off-by: Mans Rullgard <mans@mansr.com>

Subtracting a (positive) value from the address of an array violates
C99 section 6.5.6:

  If both the pointer operand and the result point to elements of the
  same array object, or one past the last element of the array object,
  the evaluation shall not produce an overflow; otherwise, the
  behavior is undefined.
Signed-off-by: Mans Rullgard <mans@mansr.com>

Subtracting a (positive) value from the address of an array violates
C99 section 6.5.6:

  If both the pointer operand and the result point to elements of the
  same array object, or one past the last element of the array object,
  the evaluation shall not produce an overflow; otherwise, the
  behavior is undefined.
Signed-off-by: Mans Rullgard <mans@mansr.com>

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

Various socket functions expect a pointer to socklen_t which is not
necessarily int.
Signed-off-by: Mans Rullgard <mans@mansr.com>

getsockname() takes a pointer to socklen_t which is not necessarily
int.
Signed-off-by: Mans Rullgard <mans@mansr.com>

HAVE_THREADS is set in config.h if pthreads or w32threads is
available, which presumably the proper condition here.

Also fixes undefined behaviour in preprocessor directives.
Signed-off-by: Mans Rullgard <mans@mansr.com>

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

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

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

This avoids Doxygen believing the attribute is the function name.

Fixes CID602232.

It is wrong to force -mcpu=ultrasparc when checking for vis.
Signed-off-by: Mans Rullgard <mans@mansr.com>

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

This makes sure the previously always installed public header
lzo.h is installed if the LZO functionality is enabled.
Signed-off-by: Martin Storsjö <martin@martin.st>

This was previously implicitly done in ff_raw_read_header().
Fixes fate tests with pcm input.

avio_size() reports the filesize which returns 0 for /dev/null.
avio_tell() reports the current position.

Also handle errors from avio_tell().

Fixes CID700635, CID700636 and CID732274.

It is not used anymore and is a kludge.

The code shared is not actually shared with anything else.

ff_raw_read_header is used only for this demuxer for video.

The function is used elsewhere and does not belong with the LZO code.

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