[vlc-devel] Re: vlc: svn commit r18218 (courmisch)

[Sigmund Augdal]

On 12/3/06, Rémi Denis-Courmont <rem at videolan.org> wrote:
>
> Le dimanche 3 décembre 2006 16:58, Sigmund Augdal Helberg a écrit :
> > On 12/3/06, Subversion daemon <svn at videolan.org> wrote:
> > > r18218 | courmisch | 2006-12-03 15:23:40 +0100 (Sun, 03 Dec 2006) |
> > > 4 lines
> > > Changed paths:
> > >    M /trunk/src/Makefile.am
> > >    M /trunk/src/input/input.c
> > >    M /trunk/src/input/input_internal.h
> > >    A /trunk/src/input/vlm.c (from /trunk/src/misc/vlm.c:18216)
> > >    M /trunk/src/misc/modules.c
> >
> > Was this change intentional? Shouldn't it be mentioned in the
> > changelog?
>
> Uh? The modules.c change was an accident.


Let me guess: You were using valgrind and had issues resolving the
symbols of the unloaded plugins?
Perhaps we should make an option for this, or make it the default when
vlc is configured with --enable-debug?

Sigmund

>  Also can you say a few words on when/how this memleak happened and
> > an approximation on how log it's been around?
>
> I'm afraid no. I am not bored enough to look this up.
>
> > And finally, shouldn't p_input->p get a proper name?
>
> Probably (but *I* won't do it).
>
> --
> Rémi Denis-Courmont
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20061203/4ce8cdd5/attachment.html>