Re: [FFmpeg-devel] [PATCH 2/2] hwcontext_vulkan: fix make checkheaders fail
On Sat, May 30, 2020 at 10:23 PM James Almer wrote: > > On 5/30/2020 11:19 AM, James Almer wrote: > > On 5/30/2020 10:16 AM, Jun Zhao wrote: > >> From: Jun Zhao > >> > >> make checkheaders will get error as follow: > >> CC libavutil/hwcontext_vulkan.h.o > >> In file included from libavutil/hwcontext_vulkan.h.c:1: > >> ./libavutil/hwcontext_vulkan.h:130:23: error: ‘AV_NUM_DATA_POINTERS’ > >> undeclared here (not in a function) > >> 130 | void *alloc_pnext[AV_NUM_DATA_POINTERS]; > >> | ^~~~ > >> ./libavutil/hwcontext_vulkan.h:199:43: warning: ‘enum AVPixelFormat’ > >> declared inside parameter list will not be visible outside of this > >> definition or declaration > > Also include pixfmt.h while at it. Don't rely on frame.h including it. > > >> > >> Signed-off-by: Jun Zhao > >> --- > >> libavutil/hwcontext_vulkan.h | 2 ++ > >> 1 file changed, 2 insertions(+) > >> > >> diff --git a/libavutil/hwcontext_vulkan.h b/libavutil/hwcontext_vulkan.h > >> index aba9833..0f6a51f 100644 > >> --- a/libavutil/hwcontext_vulkan.h > >> +++ b/libavutil/hwcontext_vulkan.h > >> @@ -21,6 +21,8 @@ > >> > >> #include > >> > >> +#include "libavutil/frame.h" > > > > #include "frame.h" > > > > This file is already inside libavutil. > > > >> + > >> /** > >> * @file > >> * API-specific header for AV_HWDEVICE_TYPE_VULKAN. > >> Include "pixfmt.h" and "frame.h" as the commets in patch v2, tks ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH 2/2] hwcontext_vulkan: fix make checkheaders fail
May 30, 2020, 14:16 by mypopy...@gmail.com: > From: Jun Zhao > > make checkheaders will get error as follow: > CClibavutil/hwcontext_vulkan.h.o > In file included from libavutil/hwcontext_vulkan.h.c:1: > ./libavutil/hwcontext_vulkan.h:130:23: error: ‘AV_NUM_DATA_POINTERS’ > undeclared here (not in a function) > 130 | void *alloc_pnext[AV_NUM_DATA_POINTERS]; > | ^~~~ > ./libavutil/hwcontext_vulkan.h:199:43: warning: ‘enum AVPixelFormat’ declared > inside parameter list will not be visible outside of this definition or > declaration > > Signed-off-by: Jun Zhao > --- > libavutil/hwcontext_vulkan.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavutil/hwcontext_vulkan.h b/libavutil/hwcontext_vulkan.h > index aba9833..0f6a51f 100644 > --- a/libavutil/hwcontext_vulkan.h > +++ b/libavutil/hwcontext_vulkan.h > @@ -21,6 +21,8 @@ > > #include > > +#include "libavutil/frame.h" > + > /** > * @file > * API-specific header for AV_HWDEVICE_TYPE_VULKAN. > -- > 2.7.4 > LGTM, tnx. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH 2/2] hwcontext_vulkan: fix make checkheaders fail
On 5/30/2020 11:19 AM, James Almer wrote: > On 5/30/2020 10:16 AM, Jun Zhao wrote: >> From: Jun Zhao >> >> make checkheaders will get error as follow: >> CC libavutil/hwcontext_vulkan.h.o >> In file included from libavutil/hwcontext_vulkan.h.c:1: >> ./libavutil/hwcontext_vulkan.h:130:23: error: ‘AV_NUM_DATA_POINTERS’ >> undeclared here (not in a function) >> 130 | void *alloc_pnext[AV_NUM_DATA_POINTERS]; >> | ^~~~ >> ./libavutil/hwcontext_vulkan.h:199:43: warning: ‘enum AVPixelFormat’ >> declared inside parameter list will not be visible outside of this >> definition or declaration Also include pixfmt.h while at it. Don't rely on frame.h including it. >> >> Signed-off-by: Jun Zhao >> --- >> libavutil/hwcontext_vulkan.h | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/libavutil/hwcontext_vulkan.h b/libavutil/hwcontext_vulkan.h >> index aba9833..0f6a51f 100644 >> --- a/libavutil/hwcontext_vulkan.h >> +++ b/libavutil/hwcontext_vulkan.h >> @@ -21,6 +21,8 @@ >> >> #include >> >> +#include "libavutil/frame.h" > > #include "frame.h" > > This file is already inside libavutil. > >> + >> /** >> * @file >> * API-specific header for AV_HWDEVICE_TYPE_VULKAN. >> > ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH 2/2] hwcontext_vulkan: fix make checkheaders fail
On 5/30/2020 10:16 AM, Jun Zhao wrote: > From: Jun Zhao > > make checkheaders will get error as follow: > CClibavutil/hwcontext_vulkan.h.o > In file included from libavutil/hwcontext_vulkan.h.c:1: > ./libavutil/hwcontext_vulkan.h:130:23: error: ‘AV_NUM_DATA_POINTERS’ > undeclared here (not in a function) > 130 | void *alloc_pnext[AV_NUM_DATA_POINTERS]; > | ^~~~ > ./libavutil/hwcontext_vulkan.h:199:43: warning: ‘enum AVPixelFormat’ declared > inside parameter list will not be visible outside of this definition or > declaration > > Signed-off-by: Jun Zhao > --- > libavutil/hwcontext_vulkan.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavutil/hwcontext_vulkan.h b/libavutil/hwcontext_vulkan.h > index aba9833..0f6a51f 100644 > --- a/libavutil/hwcontext_vulkan.h > +++ b/libavutil/hwcontext_vulkan.h > @@ -21,6 +21,8 @@ > > #include > > +#include "libavutil/frame.h" #include "frame.h" This file is already inside libavutil. > + > /** > * @file > * API-specific header for AV_HWDEVICE_TYPE_VULKAN. > ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
[FFmpeg-devel] [PATCH 2/2] hwcontext_vulkan: fix make checkheaders fail
From: Jun Zhao make checkheaders will get error as follow: CC libavutil/hwcontext_vulkan.h.o In file included from libavutil/hwcontext_vulkan.h.c:1: ./libavutil/hwcontext_vulkan.h:130:23: error: ‘AV_NUM_DATA_POINTERS’ undeclared here (not in a function) 130 | void *alloc_pnext[AV_NUM_DATA_POINTERS]; | ^~~~ ./libavutil/hwcontext_vulkan.h:199:43: warning: ‘enum AVPixelFormat’ declared inside parameter list will not be visible outside of this definition or declaration Signed-off-by: Jun Zhao --- libavutil/hwcontext_vulkan.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavutil/hwcontext_vulkan.h b/libavutil/hwcontext_vulkan.h index aba9833..0f6a51f 100644 --- a/libavutil/hwcontext_vulkan.h +++ b/libavutil/hwcontext_vulkan.h @@ -21,6 +21,8 @@ #include +#include "libavutil/frame.h" + /** * @file * API-specific header for AV_HWDEVICE_TYPE_VULKAN. -- 2.7.4 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".