[FFmpeg-devel] [PATCH] Properly check for sys/resource.h

Ramiro Polla ramiro
Thu May 8 02:49:21 CEST 2008


Diego Biurrun wrote:
> On Wed, May 07, 2008 at 09:55:47AM +0100, Ramiro Polla wrote:
>> Diego Biurrun wrote:
>>> On Wed, May 07, 2008 at 09:32:19AM +0100, Ramiro Polla wrote:
>>>> $subj
>>>>
>>>> --- ffmpeg.c	(revision 13058)
>>>> +++ ffmpeg.c	(working copy)
>>>> @@ -36,7 +36,9 @@
>>>>  
>>>> -#if !defined(HAVE_GETRUSAGE) && defined(HAVE_GETPROCESSTIMES)
>>>> +#ifdef HAVE_SYS_RESOURCE_H
>>> Has the check for getrusage become redundant now?
>> Kind of... Having sys/resource.h doesn't imply having getrusage(). I 
>> suppose the getrusage check could be removed afterwards. But then we 
>> call getrusage() under ifdef HAVE_SYS_RESOURCE_H?
> 
> Apparently it is not redundant.  So I guess the patch is OK.

Applied.

Ramiro Polla




More information about the ffmpeg-devel mailing list