[FFmpeg-devel] [PATCH] qt-faststart: Remove unnecessary checks before calling free
Måns Rullgård
mans
Sat May 1 13:34:00 CEST 2010
Alexander Strange <astrange at ithinksw.com> writes:
> On May 1, 2010, at 7:07 AM, M?ns Rullg?rd wrote:
>
>> Martin Storsj? <martin at martin.st> writes:
>>
>>> On Fri, 30 Apr 2010, Frank Barchard wrote:
>>>
>>>> On Fri, Apr 30, 2010 at 11:16 AM, Baptiste Coudurier <
>>>> baptiste.coudurier at gmail.com> wrote:
>>>>
>>>>> On 04/30/2010 03:39 AM, Martin Storsj? wrote:
>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> $subj
>>>>>>
>>>>>>
>>>>> Well I'd be ok, but I thought there were some systems that would
>>>>> not handle free(NULL) out there.
>>>>
>>>>
>>>> Agreed.
>>>
>>> Hmm, ok. Out of curiosity, what systems do we support where free(NULL)
>>> isn't ok?
>>
>> None. I have never seen such a system. They are a myth perpetrated
>> by people who like justifying extraneous if statements.
>
> C99 requires free(NULL) to work.
Knowing that, I assumed we were talking about not-quite-C99 systems,
of which there are many, yet even amongst these I have not seen one
where free(NULL) does not work.
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list