The check was stupid (always was). This seems way more logical.

- This is REALLY slow.
- It is slow each time you Seek to somewhere that you have not been/played before.
- We ought to create a "Repair index" function instead, like for avi. (no time before release)

This close ticket #1687

Fix compilation warnings (here the lib might have ask for const char* instead of char as they don't modify the string at all)

Fix a crash when exiting with simple pref open.

macosx: our crashreport-mail needs to be UTF8 if we want to use NSUTF8StringEncoding for string conversation ;)

plus some minor layout changes to make sure that all localised string fit in our UI

Spotted by thedj with the help of our bugreport ml.

Signed-off-by: Derk-Jan Hartman <hartman@videolan.org>

Fix #ifdef XP_UNIX && !defined( __APPLE__), it should be #if defined(XP_UNIX) && !defined( __APPLE__)