[FFmpeg-devel,1/4] avcodec/mediacodec_wrapper: add missing <libavcodec/avcodec.h> include

Submitted by Matthieu Bouron on July 4, 2019, 2:11 p.m.

Details

Message ID 20190704141138.GA3628@tsuri
State Accepted
Commit 6251ad89a77566254b934fbf95159d66e29328a8
Headers show

Commit Message

Matthieu Bouron July 4, 2019, 2:11 p.m.
On Thu, Jul 04, 2019 at 04:03:42PM +0200, Nicolas George wrote:
> Matthieu Bouron (12019-07-04):
> > ---
> >  libavcodec/mediacodec_wrapper.h | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/libavcodec/mediacodec_wrapper.h b/libavcodec/mediacodec_wrapper.h
> > index f0de16d669..58e5dc7d39 100644
> > --- a/libavcodec/mediacodec_wrapper.h
> > +++ b/libavcodec/mediacodec_wrapper.h
> > @@ -26,6 +26,8 @@
> >  #include <stdint.h>
> >  #include <sys/types.h>
> >  
> 
> > +#include <libavcodec/avcodec.h>
> 
> AFAIK, this is not the correct way of including one of our own headers,
> especially from the same sub-library. Just "avcodec.h".

Fixed in attached patch.
Thanks.

Patch hide | download patch | download mbox

diff --git a/libavcodec/mediacodec_wrapper.h b/libavcodec/mediacodec_wrapper.h
index f0de16d669..b106ff315a 100644
--- a/libavcodec/mediacodec_wrapper.h
+++ b/libavcodec/mediacodec_wrapper.h
@@ -26,6 +26,8 @@ 
 #include <stdint.h>
 #include <sys/types.h>
 
+#include "avcodec.h"
+
 /**
  * The following API around MediaCodec and MediaFormat is based on the
  * NDK one provided by Google since Android 5.0.