This targets the same issue as 5b8a4057, but in a saner approach

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

items stored in a xib should only interacted with after its loaded, which may be after NSApp is

This is more coherent with the rest of the worlds...

AppKit could be accessed by a non-main-thread

extras/package: disable the macosx dialog provider in VLC.app release builds, since it should only be used by VLCKit

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

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Also inline str_format().

There are several issues with str_format() and str_format_meta():
 - the current playlist item is used rather than the actual input,
 - the playlist gets activated for nothing,
 - they assume the output does not outlive to the input (item).

This mostly reverts commit cccd51a5.

pl_Get() can only be used in interfaces, playlist and ML code.

This could crash. The playlist needs to be started (at least until
someone resolves bug #5460).

ml_AttributeIsString() defines String as 1 and Integer as 0.
The switch/case was just doing the opposite...

vlclua_playlist_add_internal() requires the playlist only if there is
no parent item. There is always a parent item when demuxing.

pl_Get() has side effects.