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

We generally do not declare variables within for statements and
there are compilers that choke on such constructs.

Fixes Bugzilla #279.
Signed-off-by: Diego Biurrun <diego@biurrun.de>

Also remove one pointless zero initialization in rangecoder.c.

When using "-f x11grab -i :0.0" valgrind reports a definitely lost
memory block with this message:

==31544== 5 bytes in 1 blocks are definitely lost in loss record 1 of 2
==31544==    at 0x4026E68: memalign (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==31544==    by 0x4026F17: posix_memalign (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==31544==    by 0x60D399A: av_malloc (in /usr/lib/x86_64-linux-gnu/libavutil.so.51.22.1)
==31544==    by 0x60D3A70: av_strdup (in /usr/lib/x86_64-linux-gnu/libavutil.so.51.22.1)
==31544==    by 0x4A2BE58: ??? (in /usr/lib/x86_64-linux-gnu/libavdevice.so.53.2.0)
==31544==    by 0x506D29E: avformat_open_input (in /usr/lib/x86_64-linux-gnu/libavformat.so.53.21.0)
==31544==    by 0x400A80: main (in /home/ao2/WIP/am7xxx-play/tests/a.out)

The 5 bytes lost are the ones from param = av_strdup(":0.0"), so let's
free param in the exit path.

Also check the av_strdup() return value.

Note: calling av_free(param) even when av_strdup() fails and param is
NULL is OK and keeps the code simpler without adding another label to
skip av_free().
Signed-off-by: Ronald S. Bultje <rsbultje@gmail.com>

Signed-off-by: Diego Biurrun <diego@biurrun.de>

Remove some unneeded memsets.

Instead of busy waiting use poll();

Let pass the codec name to -pixel_format and introduce -input_format.

V4L2_FMT_FLAG_EMULATED was added in 2.6.32.

Make use of the experimental framesize enumeration ioctl if available.

Unbreak direct streamcopy.

Check capabilities directly in the function, further simplify the code.

Only mmap is supported.

Video4linux2 supports both interlaced and non-interlaced mode, do not
ask for interlaced if not necessary.

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

It's supposed to be called only from (de)muxers.

It makes more sense for a bit mask to use an unsigned type.
The change should be source and binary compatible on all
supported systems, hence micro version bump.

Fixes a few invalid shifts.
Signed-off-by: Mans Rullgard <mans@mansr.com>

The frame duration was calculated without taking in account the
bytes per sample.

Thanks to Lorenzo Pistone <blaffablaffa@gmail.com> for pointing
the issue and providing an initial fix.

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

All current usages of it are incompatible with localization.
For example strcasecmp("i", "I") != 0 is possible, but would
break many of the places where it is used.

Instead use our own implementations that always treat the data
as ASCII.
Signed-off-by: Martin Storsjö <martin@martin.st>

It currently use the simple api and is using the latency information
provided only to offset the stream start.
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

It's only used in the JACK device.

Fixes linking shared lavd with JACK enabled.

They are used in libavdevice.