This reverts commit 8b49a93d.

Fix #8124

(cherry picked from commit 45032ec3b271641aaa3d9613e7c579fabc3d475a)

Conflicts:
	modules/codec/avcodec/audio.c

Signed-off-by: Christoph Miebach <christoph.miebach@web.de>

(cherry picked from commit 8cbd52ac8a500a96378a35f74be12c6d73b50d18)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Last bump was in 2.0.2, possibly omitted since then.
(changed by commit 6be4754f)

(changed by commit 2c27b625)

autogen.sh does not exist anymore
(cherry picked from commit b3e62f036664bf4edc1d1edafd3b9df012c92ffa)
Signed-off-by: Rafaël Carré <funman@videolan.org>

This avoids the infamous VLC playlist bug whereby the playlist stops if
the current item is deleted. It turns out PulseAudio tends to "update"
a source right after VLC opens it, and then VLC stopped playing almost
as soon as it started.
(cherry picked from commit 5678c1900b38babcbdf453fcbf3ee4f9a863e594)

Based on the gnutls_handshake manual the function must be called again
until it returns 0 (or a fatal error).
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>
(cherry picked from commit ca82afbfb7ee9a083d13c7861408e5759f2509cd)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Closes #8076
(cherry picked from commit 2c1b60aec7e7fbeb4d0f0007c1933ad1572af490)
Signed-off-by: Sébastien Escudier <sebastien-devel@celeos.eu>

If foo[i] or bar[i_index] are used in TAB_REMOVE / TAB_FIND, they will
use another variable with local scope declared inside the macro.

for (int i = 0; i < 1; i++) {
    TAB_REMOVE( count, array, foo[i] );
}

will expand to:

for (int i = 0; i < 1; i++) {
    int i_index;
    .....
    for (int i = 0; i < count; i++)
        if (array[i] == foo[i])
        {
            index = i;
            break;
        }
    .....
}

And inner scope i is used to index foo, instead of the outer scope i
which we would expect without knowing the content of the macro.
(cherry picked from commit 728ef39d15cdbfc5b1bc9beba8a97493ef6c08fc)
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Signed-off-by: Christoph Miebach <christoph.miebach@web.de>

Should close #8125
(cherry picked from commit 9daa9e5d25ebebc35a8b6be32c3b357e9f1985f9)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

The dummy support patch also fix #5287.
(cherry picked from commit f000f5d9a46db78bf8e58459b65130e2a59568a1)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

macosx: work-around for Mountain Lion which treats folder with the name 'BDMV' and the sub item 'INDEX.BDM' as a file with the UTI 'public.directory'
(cherry picked from commit 79c8560a5e36d6f872a740aeeb6a62c8069c3eae)

Conflicts:
	modules/gui/macosx/open.m

Surprisingly, my gcc did not generate a possible unitilization warning.
(cherry picked from commit ce96ee78f58c6d24eb66d5544b571cc3d0aa4343)

 * Add missing call to vlc_object_release() in MarshalMetadata
 * Separate code paths with empty lines
 * Fix bracket position in if blocks
(cherry picked from commit 3ed48e03b85431f8024389f1b352854f0714ec3c)
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

VLC was getting kicked from the D-Bus when a track was stopped, because
it was generating invalid data on the wire when sending the
PropertiesChanged signal for the Metadata property.

The issue was that if there was now no current track, GetInputMeta would
never be called and the the variant would never be populated with the
"a{sv}" structure that the call to dbus_message_iter_open_container
claimed it would be.

We now share the code that GetProperties used, which dealt with this
correctly, although now both use CurrentInput (which is what the signal
previously used) instead of CurrentPlayingItem (which is what
GetProperties previously used).
Signed-off-by: Mirsal Ennaime <mirsal@videolan.org>
(cherry picked from commit 5460e18e20719255d4ed1461594fe72530a19f85)
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Rate should never be 0.0 (according to the spec), so we pick a sensible
default when we do not have a current track: 1.0.

Also, when the current item changes, the Rate may also change.  So add
it to the PropertiesChanged signal just to be sure.
Signed-off-by: Mirsal Ennaime <mirsal@videolan.org>
(cherry picked from commit 8f754d8715c72abc4ba1b0f26ae136c1db41437b)
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Several of these entries, such as xesam:artist, should be lists of
strings, not strings.
Signed-off-by: Mirsal Ennaime <mirsal@videolan.org>
(cherry picked from commit ad8039e3e3b910730e71921b9f8952107644695a)
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Signed-off-by: Christoph Miebach <christoph.miebach@web.de>

Signed-off-by: Christoph Miebach <christoph.miebach@web.de>

Signed-off-by: Christoph Miebach <christoph.miebach@web.de>

Signed-off-by: Christoph Miebach <christoph.miebach@web.de>

Signed-off-by: Christoph Miebach <christoph.miebach@web.de>

Signed-off-by: Christoph Miebach <christoph.miebach@web.de>

Signed-off-by: Christoph Miebach <christoph.miebach@web.de>