postproc: fix potential crash (var_DelCallback missing and callback added too
early).

Cosmetics (no functionnal changes).

Missing var_DelCallback.

Don't ignore DTS in wav.

Decode correctly some ac3 in wav files.

xspf: Fix a crash on track elements without a location

A <track> can have 0 or more <location> elements. VLC doesn't like elements without a uri however, so init an element with vlc://nop. Note also that we currently only support 1 location element per track....

This closes #2891

Decode G.726 in Wav

Qt: make open dialog modal.

This ensure consistency with simple Open.
Moreover, with those KDE effects, this is cool.

Move the add/del_callback call (to be sure to have the needed ressources).

Qt: fix remaining remeber-folder bug.

Close #2851
(cherry picked from commit 084a6247a99e90c1ab66090811f2a96f8c816ed7)

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

dv: fix compiler warnings about unused variables.
(cherry picked from commit 31d64379a5706513ee6ed4e440c9e723a9842c9e)

Signed-off-by: Jean-Paul Saman <jean-paul.saman@m2x.nl>

dv: destroy mutex after vlc_pthread_join()
(cherry picked from commit f23f5e62dcb29ca9b12a1cac73a00c464dbe6f3d)

Signed-off-by: Jean-Paul Saman <jean-paul.saman@m2x.nl>

Cleaning/factorize.

samba: remove debug test.

Rewrite the samba access module whithout samba context (not needed) and using
only non-deprecated functions.

Fix warning (the parameter is an integer, not a pointer)

RTP: fix sdp=file:// with mux

RTP/PS: fix crash at exit

(Rtp)Add then MuxAdd, so MuxDel before (Rtp)Del. Indeed, the PS muxer
sends packets when it's deleted, so the RTP stream is still needed.

directory: handle relative paths properly

Typo.

Fix a segfault (buffer overflow for win32 only).

Fix memleak when using --demux.

Cosmetics.

Cleaning.

input_item_SetURI(): add a NDEBUG printf check for paths.

When it is called with strings that contain " " or '\' or don't have ://, we have an indication that the caller is likely providing a path. This will help us find these cases in the future.

Improve x11 xinerama fullscreen window placement

Select fullscreen display the one that has most of the video-window,
then you get qt4 fullscreencontroller on same display also, and it's
more logical (imo). Algorithm is prettymuch the same as Nigel Tao has
made in his gpl2 project superswitcher, Didn't want to reinvent this
with different variable-names ;)

input: SetURI No longer use DIRSEP. We are dealing with URIs, not paths.

macosx: Ignore attachment:// in Art uri for now.

macosx: Just use openWithURL, instead of uri->path

libvlc: Abort in case of unhandled exception.

skins2: solve crashes due to releasing variables at termination

skins was using two lists of variables (named and anonymous).
The problem was that variables from one list held references to variables from the other list and vice-versa. Whatever the order of releasing them, crashes could occur.

This patch uses the anonymous list to keep a reference on **all** variables.
This guarantees they are released in the reverse order from creation.

When we fill the name of an input item with URI parts, use decode_URI() so it becomes easier to read.

Do not inherit xspf-open demux from media library

XSPF: remove harmful base decoding

Assume input_item_t->psz_uri contains an URI

, like the name indicates.
Pass unmodified URIs from xspf to vlc, and stores them in xspf
unmodified as well

This commit partly reverts [9a746cfa3078c53eed57d2102002b39c283c6ab4]
Code using psz_uri should be checked for consistency, and GUIs might want to decode components at display

playlist_MLLoad: format library URI correctly

playlist_Import: use make_URI

Excessive white space

string: missing space

Adding a libvlc_media_add_option_flag

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Typo.

skins2: ensure the new SubX default skins doesn't crash on exit

SubX crashed on both Linux and Windows in a set::erase call (STL) when releasing C++ objects. Swapping two loops solved it. (the second loop was referencing variables released in the first loop). This makes SubX terminate nicely, but doesn't solve all crashes for skins2. (more cleanup to do)

macosx: memleak due to unneeded strdup()

rotate filter: Avoid a crash

This fixes a DIVIDEBYZERO crash in the rotate filter. Does not seem like the right solution to me, but dionoea should probably look into that, since he wrote most of this filter.

This crash occured with http://streams.videolan.org/issues/HIGNFY%20S25E02%20-%20Part%201.flv

plane 1 (non Y-plane)
i_visible_lines = 119
p_pic->p[Y_PLANE].i_visible_pitch = 320
p_pic->p[Y_PLANE].i_visible_lines = 239
i_visible_pitch = 160

macosx: NSScreen(VLCAdditions) does not leak.

The Objective-C runtime will properly unload this at the end of the application. The mozilla plugin should not even be using this at all.

Revert "macosx: NSScreen dealloc is not only called at end of execution of course."

This was even worse. +load() != init()
This reverts commit b5feb899d6047e2371ce18fe7d73c51fe270e2a1.

macosx@core: removed the usage of dyld's 'Library Functions' completely

these are automagically 'replaced by the more efficient 'Dynamic Loader Compatibility Functions.'' as suggested by Apple's documentation. IMO, this should be ported to the 1.0-bugfix branch as it cleanly fixes some 64bit crashes and should improve the performance slightly

macosx: NSScreen dealloc is not only called at end of execution of course.

This fixes the NSScreen crash, which could be accurately reproduced by following the steps in #2880.

panoramix filter needs xcb-randr 1.3

Revert "xcb: as we require version 1.3, let's at least test in configure for it too."

This reverts commit 4a6e2de24341848c5592bf67ad4377d624d6f997.

Remove vlc_UrlEncode

Remove non-existent target

Simplification.

podcast: save configuration when adding/removing podcast from the dialog.

skins2(SubX): correct a typo ("notdvd.isActive" => "not dvd.isActive" )

Closes #2875 : relative paths in xspf playlists

copy paste mistake in [7f43b9342a40189805621c02345539e42eae4c94]

macosx plugin: Set miminum system version for the mozilla plugin as well.

macosx: Set minimum OS version for the VLC application bundle

This works with 10.2, 10.3 and 10.4, but 10.3 requires the code we have in VLCApplication.

macosx: Update list of extensions. Fix video.icns ->  movie.icns
(cherry picked from commit 8b9d73a700bf003be654750df29f5af30f854bfd)

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

Cosmetics in live555: Factor out duplicate code, avoid ``? True : False''.

Split the test for user/passwd and the test for the port number (the two tests are independent).

Signed-off-by: JP Dinger <jpd@videolan.org>

Fix the previsous commit: if the user or the password is set in the url, use
it.

These header files are global and not local.

Cleaning.

Remove duplicate code.

Add missing CoUninitialize.
For each CoInitialize we have to call CoUninitialize.

FIx a segfault: we need both user and password to be not NULL.

mac text renderer: Fix a nasty crash in the YUVA renderer

This really fixes #2649
This entire module needs some refinement, but since we are gonna have to dump it for CoreText very soon, there is little point in doing that.

mac text renderer: make sure our picture is of the right height.

This will likely fix #2649

xcb: as we require version 1.3, let's at least test in configure for it too.

Signed-off-by: JP Dinger <jpd@videolan.org>
(If at all possible, we should reduce the requirements instead.)

Qt: use the native dialog for Save Playlist

This is better for translations, and look'n feel.
This defaults the name to .xspf, if not a valid extension is provided.

Qt: reinventing the wheel is always a good solution.

Qt: be sure to use everywhere the saved path.

And also fix the support for it on Windows.

Simplification.

Factorize.

Fix potential memleak.

Dead initialization.

New default skin: SubX

Signed-off-by: Christophe Mutricy <xtophe@videolan.org>

I like the stories well enough but I can't get myself to like this as a codename for 1.1. xtophe suggested sin city, so with a bit of imdb assistance, we have...

http interface: strdup memleaks

Qt: store the filedialog path in Qt setings, instead of vlc config (internal option, btw)

This simplifies a bit the code, should solve the issue reported over and over by the users.

Qt: first step to fix directory remembering.

Qt: cosmetics and comments.

Qt: correct right click selection.

That way, when you right-click on the not-selected item, you can delete it.

Chorus filter in NEWS.

Fix compilation (thanks to cppcheck developpers to notice this bug)

Fix memleak.

Never print the password in the logs.

Don't print the password in the logs (even debug ones).

Remove dead increment.

Missing const.

Remove dead initialization.

Fix #2720 for master also. Finally that's not an hack as the code is build to
stop the playlist just after removing the playing item (so that's ok to stop
the PL even when it's empty).

(cherry picked from commit be1f4305f1203ccf5acc402fd2cc80dde3f5b1cd)

Signed-off-by: Rémi Duraffort <ivoire@videolan.org>

Use vlc_mutex_locker

Use vlc_mutex_locker.

No need to test for NULL here.

No need to test for p_sys != NULL as it's already tested in the constructor.

Remove unused includes and factorize.

Fixed a typo.