Fixes a bug introduced on [795fa06c]
Closes #2794
(cherry picked from commit fc9987fa29409be3cae9743dbae265959477747c)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

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

* initmethods returning nil. checks moved outside init.
* leaking module_config_get. Keep better track of those.
* spurious strdup
* leaked hotkeys NSMutableArray when resetting the preferences.
(cherry picked from commit 27d841889c9116d46be643fa482f9ca562d233f3)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

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

Introduced by [ca17039cd8dbaff98d0d4f08b94d3f3efa92af98]
(cherry picked from commit c543a40de86224b961a089d0a17374867f835da4)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

According to x264 code, if it fails, it return NULL.
And with a first NULL parameter x264_encoder_headers() segfaults.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>
(cherry picked from commit 77dea4efed8f1ce11edf551ac33b53ead9fb16a8)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Note that "avformat{mux=mjpeg}" is also valid.
(cherry picked from commit f69af253a4fe0dbf4121c4a320d01adb1c6d0cab)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

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

When reading a file:// URL from an xspf entry, store the decoded file path; else store the unmodified URL

Always keep album art URLs always encoded and decode the path when needed
Interfaces & plugins only use file paths
Playlist core extract attachment:// URLs (from file meta data), http:// or other (from meta data fetchers like lua) and cache them to a file

Thanks to courmisch for clarification
Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>
(cherry picked from commit a654d4a1)

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

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

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

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

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

(cherry picked from commit bc3883a52ee35d2d118f8a7c43093faa253cc0b4)

(cherry picked from commit 02b581e1)

(cherry picked from commit b5687139)

Mention that MPRIS is at version 1.0
Remove some trailing spaces
Remove unused argument 'lock' from MarshalStatus()
(cherry picked from commit f193adfb)

(cherry picked from commit bad63db868d59b72dfc48ab767e5ee74f4d5d3c6)

(cherry picked from commit feec0f30)

Fix mozilla autoconf check : AC_PATH_PROGS() was used incorrectly, and the check wouldn't error if {mozilla,seamonkey}-config or xulrunner wasn't found
(cherry picked from commit f3fe98db)

Lots of strings still missing, help would be welcome
(cherry picked from commit 946eba4a)

playlist sorting: for inline to work, the text has to be available. IE putting the function behind all others that use it isn't going to work, and marking forward declarations inline won't help fix it.

Fix compilation

(cherry picked from commit cbccd7297c241830e796f0f239b4b90d19c16c75)
Signed-off-by: Derk-Jan Hartman <hartman@videolan.org>

No need to backport this, as 1.0-bugfix only includes classic nibs

"daemon" is kept as the default, other accepted values are for now
"user" and "local0" through "local7"