[vlc-devel] [PATCH] Add a libvlc_set_video_output function

[Rémi Denis-Courmont]

Le mercredi 12 février 2014, 21:17:25 Hugo Beauzée-Luyssen a écrit :
> On Wed, Feb 12, 2014 at 8:06 PM, Rémi Denis-Courmont <remi at remlab.net> 
wrote:
> > Le mercredi 12 février 2014, 19:33:10 Hugo Beauzée-Luyssen a écrit :
> >> I completely agree with this. Yet again, that does not suit what I'm
> >> looking for. More specifically, I'm interested in the video infos,
> >> fps, dimensions, ... which are only accessible once the media
> >> (including the video, obviously) is playing.
> > 
> > No. It is accessible after the media was preparsed. There is no need to
> > play at all.
> 
> True.
> My bad, I forgot to mention that I need to take a snapshot as well,
> which requires a vout.

Sorry but this is reaching absolutely stupid and ridiculous proportions.

You are trying to add a function that brings zero added value over 
libvlc_new() for its nominal use case (hiding video), that is really just a 
hack (since it would make much more sense to disable video completely), 
because you want to do hardly related things with the media player.

No. LibVLC is enough bloated and confusing as it is, I do not want that kind 
of hacks. There has to be a sanity limit somewhere and this would be an 
absolutely terrible precedent.

Besides, you can already dump the video frames with the video callbacks and 
get the track meta data. Otherwise, this belongs in streaming output (not to 
mention gstreamer).

-- 
Rémi Denis-Courmont
http://www.remlab.net/