diff mbox

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

Message ID 1469754608-85489-1-git-send-email-josh@itanimul.li
State Accepted
Commit 2068d116db9883c71cb00b76e7d371cc30164d08
Headers show

Commit Message

Josh Dekker July 29, 2016, 1:10 a.m. UTC 
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(-)

Comments

Timothy Gu July 29, 2016, 2:49 a.m. UTC | #1 
On Thu, Jul 28, 2016 at 6:11 PM Josh de Kock <josh@itanimul.li> 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
Josh Dekker July 29, 2016, 8:20 a.m. UTC | #2 
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
diff mbox

Patch

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"