[vlc-devel] Re: vlc: svn commit r7902 (fenrir)

[Laurent Aimar]

On Sun, Jun 06, 2004, Olivier Aubert wrote:
> >  Btw, the module uses input_Tell that is deprecated, Is it really needed ?
> I was just following the advice that I got on this list on Nov. 2003
> (OK, 6 months ago, but I have been reading the list since and did not
> notice any official deprecation announcement of input_Tell) :
> http://article.gmane.org/gmane.comp.video.videolan.vlc.devel/7410
> 
> I will remove it. But it would be very nice if the People Who Know would
> mark deprecated functions as such in their comments, so that we can
> follow what is Right and Wrong. For the moment, the code itself does not
> have anything about that, and the developer documentation just says "The
> most important file is include/input_ext-intf.h, which you should know
> almost by heart." A simple comment like "Deprecated. Use ???? instead."
> would be extremely useful.
 Yes, the developer doc isn't updated (I never update it myself) :((
So yes, for people starting on vlc it's not easy.
 If you have any questions, just ask them, vlc-devel@ is here for that,
you can also come to irc, a lot of developer are here

Anyway, about the input, I plan to remove all deprecated functions as
well as access1/demux1 support very soon (within 1-3 weeks), and I
will try to update the docs.

> On the other hand, what I was looking for was a way to add timestamps to
> the snapshots. Using information for p_input results anyway in some
> error (as the image is displayed some time after being read). So it is
> not perfect even with the var_Get (p_input, "time") solution. If
> somebody has an idea about getting this information, please tell me.
 You can try var_GetTime( p_input, "time" ) - p_input->i_pts_delay, it
should be more accurate, but I'm not that sure.

-- 
fenrir

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html
If you are in trouble, please contact <postmaster at videolan.org>