[vlc-devel] [PATCH] libvlc: help debug object references remaining in debug builds

Romain Vimont rom1v at videolabs.io
Fri Aug 17 11:47:12 CEST 2018


On Fri, Aug 17, 2018 at 11:33:44AM +0200, Thomas Guillem wrote:
> Maybe, we should bring this patch back ? https://mailman.videolan.org/pipermail/vlc-devel/2017-June/113477.html

+1, this would help a lot.

(We could just add a call to fflush(NULL) before the assert() so that
the buffers are flushed before aborting.)

> The debugger is not always attached, and sometimes you can't reproduce a leak. 
> I often use this patch to get the actual leak...
> 
> On Fri, Aug 17, 2018, at 10:59, Jean-Baptiste Kempf wrote:
> > Hello,
> > 
> > On Fri, 17 Aug 2018, at 08:41, Steve Lhomme wrote:
> > > No, before we could follow internals->first in the debugger. Now it's 
> > > [...]
> > > internals->siblings which just had garbage in it. I tried 
> > > vlc_list_foreach(), vlc_list_first_entry_or_null() no luck.
> > 
> > I would argue that we need to be able to have the list of leaked item 
> > easily, and not only with a debugger, notably for user reports;
> > but the vlc_list is fine and a good idea.
> > 
> > 
> > -- 
> > Jean-Baptiste Kempf -  President
> > +33 672 704 734
> > _______________________________________________
> > vlc-devel mailing list
> > To unsubscribe or modify your subscription options:
> > https://mailman.videolan.org/listinfo/vlc-devel
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list