When the first entry is read and has a clientskip attribute, It keeps a
backup of the playlist entry for later use. If another entry exists
without clientskip attribute, the backup will be discarded.

When finished parsing, we search for a backup, if found, we add it (it
means we did not find any entry).

Replace arbitrary sleep delays by active waiting of state change
Add state change wait where needed (after play()/next()/previous())

This reverts commit f2564c11.

...instead of the configuration. As a side effect, the command line
parameter should not be visible in the preferences, and more importantly
not be saved as part of the persistent configuration (fixes #1428)
(and really fixes older #1106).

We might be able to remove a few "dummy" change_unsaveable(). Some of
them really were just work-arounds for this bug. In principle, we could
possibly remove all of them as long as we keep add_internal().

Note that this commit will render any command line option inoperant if
it is read with config_Get*() instead of var_Inherit*() or var_*Get*().
I already fixed the most commonly used occurences, but there are some
left, especially inside plugins.

This is silly. This option cannot be set from the configuration, so it
does not beling in the preferences.

This fixes #3168. The audio_format plugin was ignorantly claiming to
convert the sample rate and channel count, though it obviously did not.

This fixes the following bug:
When scrolling the mouse wheel down, volume jumps from 0% to 400%.

In most cases, there is enough space for the 2 old samples in the
input buffer head room. In other cases, we anyway need to memory copy
the whole buffer. So we now use block_Realloc(). This also saves us from
copying every samples when resampling.

Unfortunately, the transcode plugin seems to be feeding crap into the
resampler, thus it still crashes.

Also fix a use after free

POSIX sem_*() functions return -1 on error and the error code in errno
(contrary to most POSIX thread functions).

This gives a use to the otherwise useless artUrl variable
This prevents qt4 interface from consuming 100% of CPU when art image
has to be scaled from a big source picture

This reverts commit 4b5229c8.

The stream_out_t chain creation is modified: all modules are created by
the core (or by stream_out_duplicate) instead of being created by the
previous module.

sout_StreamChain{New,Delete} replace sout_Stream{New,Delete} to handle
modules chains instead of individual modules

sout_Stream{New,Delete} are still used by those new functions but made
static inside stream_output.c

Remove now unneeded psz_chain from struct sout_instance_t
Replace pointer to chain of next module by pointer to next module in
struct sout_stream_t

Example use:

vlc --sout-all input.mp4 --sout
"#duplicate{dst=transcode{vcodec=mp2v},select=es=0,dst=transcode,select=es=1}:std{...}"

(dst=transcode without acodec/vcodec is a hack to pass the encoded stream to
stream_out_standard without transcoding)

I vote that we remove this plugin now.
FFmpeg supports SIPR natively now.

Sorry