[FFmpeg-cvslog] r19742 - trunk/libavutil/internal.h

Ramiro Polla ramiro.polla
Fri Sep 4 18:49:49 CEST 2009


On Thu, Sep 3, 2009 at 1:12 PM, Ramiro Polla<ramiro.polla at gmail.com> wrote:
> On Thu, Sep 3, 2009 at 12:33 PM, Benoit Fouet<benoit.fouet at free.fr> wrote:
>> On 2009-09-03 17:27, Ramiro Polla wrote:
> [...]
>>> diff --git a/libavutil/internal.h b/libavutil/internal.h
>>> index 14ea95b..7b665a9 100644
>>> --- a/libavutil/internal.h
>>> +++ b/libavutil/internal.h
>>> @@ -249,21 +249,21 @@ if ((y) < (x)) {\
>>>  #define perror please_use_av_log_instead_of_perror
>>>  #endif
>>>
>>> -#define CHECKED_ALLOC(p, size)\
>>> +#define FF_ALLOC_OR_GOTO(ctx, p, size, label)\
>>>  {\
>>>      p = av_malloc(size);\
>>>      if (p == NULL && (size) != 0) {\
>>> -        av_log(NULL, AV_LOG_ERROR, "Cannot allocate memory.");\
>>> -        goto fail;\
>>> +        av_log(ctx, AV_LOG_ERROR, "Cannot allocate memory.");\
>>> +        goto label;\
>>>
>>
>> not related to this change, but doesn't this also miss a '\n' ?
>
> Yes, it does. Ok to just add it?

...

Trivial patch attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: endline.diff
Type: application/octet-stream
Size: 684 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20090904/99a4a672/attachment.obj>



More information about the ffmpeg-cvslog mailing list