diff mbox

[FFmpeg-devel,04/14] avformat/avidec: Remove superfluous header

Message ID 20191025180746.13725-4-andreas.rheinhardt@gmail.com
State Accepted
Commit 982539a07af72ab5bae551b92e3a63bf5cf86cf6
Headers show

Commit Message

Andreas Rheinhardt Oct. 25, 2019, 6:07 p.m. UTC
bswap.h was included since 7b114c09, yet since 3788a3c0 no explicit use
of anything from bswap.h has been made, so remove this header.

(Only AV_RL32 is used and while this might imply swapping on
big-endian systems, it is contained in libavutil/intreadwrite.h.)

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
---
(I of course know that intreadwrite includes bswap.)

 libavformat/avidec.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Michael Niedermayer Oct. 25, 2019, 8:59 p.m. UTC | #1
On Fri, Oct 25, 2019 at 08:07:36PM +0200, Andreas Rheinhardt wrote:
> bswap.h was included since 7b114c09, yet since 3788a3c0 no explicit use
> of anything from bswap.h has been made, so remove this header.
> 
> (Only AV_RL32 is used and while this might imply swapping on
> big-endian systems, it is contained in libavutil/intreadwrite.h.)
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
> ---
> (I of course know that intreadwrite includes bswap.)

will apply

thx

[...]
diff mbox

Patch

diff --git a/libavformat/avidec.c b/libavformat/avidec.c
index 1ca26968fa..5cf67a4771 100644
--- a/libavformat/avidec.c
+++ b/libavformat/avidec.c
@@ -23,7 +23,6 @@ 
 
 #include "libavutil/avassert.h"
 #include "libavutil/avstring.h"
-#include "libavutil/bswap.h"
 #include "libavutil/opt.h"
 #include "libavutil/dict.h"
 #include "libavutil/internal.h"