Re: [FFmpeg-devel] [PATCH] hapdec: remove unused memory.h include

2016-07-29 Thread Josh de Kock 
On Fri, Jul 29, 2016, at 03:49 AM, Timothy Gu wrote:
> Looks good to me. For what it's worth, the include was added in
> commit 3ee217853a6741b829a2683f49c590618891b1ab, and looks like a stray
> change.
> 
> Timothy

Thanks, applied.

Josh
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] hapdec: remove unused memory.h include

2016-07-28 Thread Timothy Gu 
On Thu, Jul 28, 2016 at 6:11 PM Josh de Kock  wrote:

> Most systems have this, so it isn't really a problem to include it
> even if it's not used, but some do not have memory.h as it is
> non-standard. Since it's unused just remove it anyway.
> ---
>  libavcodec/hapdec.c | 1 -
>  1 file changed, 1 deletion(-)
>

Looks good to me. For what it's worth, the include was added in
commit 3ee217853a6741b829a2683f49c590618891b1ab, and looks like a stray
change.

Timothy
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] hapdec: remove unused memory.h include

2016-07-28 Thread Josh de Kock 
Most systems have this, so it isn't really a problem to include it
even if it's not used, but some do not have memory.h as it is
non-standard. Since it's unused just remove it anyway.
---
 libavcodec/hapdec.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/libavcodec/hapdec.c b/libavcodec/hapdec.c
index 5a399dc..f1d44cd 100644
--- a/libavcodec/hapdec.c
+++ b/libavcodec/hapdec.c
@@ -37,7 +37,6 @@
 #include "bytestream.h"
 #include "hap.h"
 #include "internal.h"
-#include "memory.h"
 #include "snappy.h"
 #include "texturedsp.h"
 #include "thread.h"
-- 
2.7.4 (Apple Git-66)

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel