[FFmpeg-devel] [PATCH v3] lavd/decklink_common: Fix error caused by -Werror=missing-prototypes

[Michael Niedermayer]

On Sun, Jun 26, 2016 at 08:24:49PM -0400, Rick Kern wrote:
> decklink_common.cpp includes a .cpp file from the DeckLink API which fails
> to build because there are non-static functions in the included .cpp file.
> This disables the missing-prototypes error so the file can be included.
> 
> Signed-off-by: Rick Kern <kernrj at gmail.com>
> ---
>  libavdevice/Makefile | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/libavdevice/Makefile b/libavdevice/Makefile
> index 585827b..e281825 100644
> --- a/libavdevice/Makefile
> +++ b/libavdevice/Makefile
> @@ -19,6 +19,8 @@ OBJS-$(CONFIG_BKTR_INDEV)                += bktr.o
>  OBJS-$(CONFIG_CACA_OUTDEV)               += caca.o
>  OBJS-$(CONFIG_DECKLINK_OUTDEV)           += decklink_enc.o decklink_enc_c.o decklink_common.o
>  OBJS-$(CONFIG_DECKLINK_INDEV)            += decklink_dec.o decklink_dec_c.o decklink_common.o

> +$(SUBDIR)decklink_common.o: CXXFLAGS += -Wno-error=missing-prototypes

Is this portable ?
is there some posix document or similar that says that this
is supported by all compilers ?

and in what directory are these system headers that cause the problem?
is this directory a system header directory ?
what compiler fails to suppress warnings from system headers ?

also:
http://clang.llvm.org/docs/UsersManual.html#controlling-diagnostics-in-system-headers

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160627/41fe841d/attachment.sig>