GitLab

This reverts commit 74cf34e6.

There is enough sucking code.

Why call vlc_thread_ready when the other side does not wait anyway...

(The clean up path of vout is in a need of more cleaning).

Correctly free spu and filter chain.
Do not free 2x vout pictures.
(At least, vlc -V x11 does not segfault any more on resize, it
just does not work)

(Cosmetic, it will allow cleaning up a bit the vout thread).

Could someone with good vout knowledge review this commit ?

attached" on vlc_object_detach although the object was attached
previously ... what could that be due to?

Fix two potential segfaults.
Cosmetics.

resizing which is horrible. Upcomming commit will fix that.

Unused video outputs are currently attached to libvlc, so there is no
point in destroying them from the playlist (twice), then from libvlc.
This should also fix #1593 (multiple vout free).

for in english code)

libvlccore: "intf-show" is now a libvlc var instead of a playlist var. This removes a vout->playlist dependency.

We shouldn't look for the playlist from the vout too... but...

This improves readability a lot and prevent us from reinventing the wheel.

Else there is a circular dependency between the vout that yield() the input, where the vout is also an input child. This should fix properly the media_instance() tests.

This sanitize the circular dependency there is between playlist and input and allow a safe destruction.