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.

Workaround buggy flv files (#2590).

 It seems that (a lot of?) flv files does not correctly set PTS,
so ignore them, with luck the decoder (avcodec) will recreate
them.
 It might break some valid files, so report any regression.

(It needs the AVCodecContext::ticks_per_frame change).

Do not warn about unsupported demux/access when aborting.

Do not ask the user if we should build the avi index when the demuxer has been killed.

Used AVCodecContext::ticks_per_frame when available.

It fixes a regression with interpolated timestamps for some
mpeg-I/II and for h264.

Cosmetics.

Fix memleak when out of memory.

Force new not to throw. Also drop a separate declaration of a static function that's called exactly once, and mark it inline instead.

Really used a weaker lock for input_ressource_HoldAout.

Used av_malloc/free for avcodec_decode_audio2 output buffer.

It is needed to ensure valid buffer alignment.

Used a weaker lock scheme for input_resource_HoldAout.

Fixed a potential race condition in input_resource_HoldVout*.

Renamed lock_vout in lock_hold in resource (No functionnal changes).

Contribs: Use $STRIP

And don't ignore errors

Update THANKS

Contribs: Fix libmpcdec configure.ac

You need a compiler.

contribs: Use autoreconf --force to avoid libtool pb.

Also use autoreconf where possible
And no need to reun automake after autoreconf

Refactor spatializer initialisation and teardown to use a table and a loop.

Revert "No need to create an object to handle the lock is such a simple case."

This reverts commit f001a156e79ba8e9c2c4f2b62330b9e155c1ec33.

This is usually a good practice. Moreover compare to locking and unlocking the
potential overhead is really small.

cosmetics (no functionnal changes).

Cosmetic (no functionnal change).

Dont install libvlc_media_list.h twice.