(cherry picked from commit 2562696be3f12bf080dea6fa6a0b690bbd761e30)

(cherry picked from commit da59205e44a1d105ad020cd9a94f8e37e6a012c7)

Blind commit

Signed-off-by: Christophe Mutricy <xtophe@chewa.net>

(See also previous commit)
(cherry picked from commit cf190075bda0c39e2a203104d9095c450e283def)

As far as I understand, this breaks C++ object scoping. qtu() creates
a temporary anonymous QByteArray from its QString parameter. That array
is destroyed at the end of the expression, thus the return value of
qtu() becomes a dangling pointer.
(cherry picked from commit 41c3dd9b229c44cb313786f96cc2089a9d5197cc)

(cherry picked from commit 00eb305f2afebf28dd649d507aca4901d0ad1e8a)

Signed-off-by: Christophe Mutricy <xtophe@chewa.net>

This caused duplciate strings in PO

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>
Signed-off-by: Felix Paul Kühne <fkuehne@videolan.org>

Pointed-out-by: Fabian Greffrath <fabian@greffrath.com> (Debian BTS)
(cherry picked from commit c5e166c05a49cacbabc52a32ee612533e909e882)

Conflicts:

	share/vlc.desktop

macosx: fixed disappearing repeat button images introduced in [8f79caa3]

Closes #4886

Unlike _endthread and ExitThread, _endthreadex doesn't close the handle.
It is therefore up to the caller to close it. Hence no need for an extra
handle duplication (that mostly amounted to a memory leak when the normal
 path was used, i.e normal return from thread with implicit _endthreadex)
(cherry picked from commit afe774e0b19c7dcad8d0ae248480df1b0660a380)
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

According to msdn doc, _endthreadex(0) is to be used when thread is
 launched with _beginthreadex().

Note that when returning from the normal path, an implicit _endthreadex is
issued anyway.  It is important to be consistent whatever the path, since
those final functions (_endthread, ExitThread, _endthreadex) don't have
the same deallocation policy.
(cherry picked from commit 99f71e5a62fdd14b031fdc89f452d873e3509c74)
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

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

Signed-off-by: Christophe Mutricy <xtophe@chewa.net>

Not working

This reverts commit f7a1db99.

macosx: fixed volume normalizer prefs and crash when saving the Growl option in combination with some config sets

(cherry picked from commit b38b9c24c20313bfa3fa76fe7dffa6177edd270e)

(cherry picked from commit cd929923ff49175a501bb3e9553a683bc42ff61c)

(cherry picked from commit bba95f3f86bf0ff241b13def7dd81c2af31829f4)

(cherry picked from commit b856ec53290ec4ed5999406552662bde0023eeb6)

Now, 1.1.9 will upgrade to 1.1.10, but 1.1.10 will act as 1.1.1 and
consider 1.1.9 newer; and not update to 1.1.11...
Of course, this commit is not good at all, but this is the simplest way
I could do it, without breaking everything.

This has probably been the most hated thing of 1.1.x, I believe.
I had to do something...
Yes, there might be issues, and leaks. Please complain ASAP to me.