[vlc-devel] Re: vlc: svn commit r19877 (jpsaman)

Jean-Paul Saman jean-paul.saman at planet.nl
Thu Apr 19 14:46:32 CEST 2007


Remi Denis-Courmont wrote:
> On Thu, 19 Apr 2007 10:33:31 +0200 (CEST), Subversion daemon <svn at videolan.org> wrote:
>> r19877 | jpsaman | 2007-04-19 10:33:30 +0200 (Thu, 19 Apr 2007) | 1 line
>> Changed paths:
>>    M /trunk/modules/access/dvb/access.c
>>
>> Check return value of malloc()
>>  > http://trac.videolan.org/vlc/changeset/19877
> 
> I am still unsure what we should do for include/vlc_arrays.h which is full
> of unchecked malloc() and realloc().

They are also used in place where we shouldn't do malloc's. So maybe we 
should go over to buffer pools in these places. Question is how large 
should such a pool be?

> Change them to macros evaluating to true/false depending on success, maybe?

Why can't it be linked lists?

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

Kind regards
Jean-Paul Saman

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html



More information about the vlc-devel mailing list