Ensure that all API headers are warning-free

3 files changed, 101 insertions, 0 deletions

diff --git a/test/test_build.c b/test/test_build.c
new file mode 100644
index 0000000..de2ed28
--- /dev/null
+++ b/test/test_build.c
@@ -0,0 +1,34 @@
+/*
+  Copyright 2020 David Robillard <d@drobilla.net>
+
+  Permission to use, copy, modify, and/or distribute this software for any
+  purpose with or without fee is hereby granted, provided that the above
+  copyright notice and this permission notice appear in all copies.
+
+  THIS SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+  ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+  WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+  ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+  OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+*/
+
+/*
+  Tests that C headers compile without any warnings.
+*/
+
+#define PUGL_DISABLE_DEPRECATED
+
+#include "pugl/gl.h"
+#include "pugl/glu.h"
+#include "pugl/pugl.h"
+#include "pugl/pugl_cairo.h"
+#include "pugl/pugl_gl.h"
+#include "pugl/pugl_stub.h"
+
+int
+main(void)
+{
+	return 0;
+}
diff --git a/test/test_build.cpp b/test/test_build.cpp
new file mode 100644
index 0000000..79f1dfc
--- /dev/null
+++ b/test/test_build.cpp
@@ -0,0 +1,36 @@
+/*
+  Copyright 2020 David Robillard <d@drobilla.net>
+
+  Permission to use, copy, modify, and/or distribute this software for any
+  purpose with or without fee is hereby granted, provided that the above
+  copyright notice and this permission notice appear in all copies.
+
+  THIS SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+  ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+  WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+  ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+  OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+*/
+
+/*
+  Tests that C++ headers compile without any warnings.
+*/
+
+#define PUGL_DISABLE_DEPRECATED
+
+#include "pugl/gl.h"
+#include "pugl/glu.h"
+#include "pugl/pugl.h"
+#include "pugl/pugl.hpp"
+#include "pugl/pugl.ipp"
+#include "pugl/pugl_cairo.hpp"
+#include "pugl/pugl_gl.hpp"
+#include "pugl/pugl_stub.hpp"
+
+int
+main()
+{
+	return 0;
+}
diff --git a/wscript b/wscript
index 2ae293d..f799ddd 100644
--- a/wscript
+++ b/wscript
@@ -443,6 +443,37 @@ def build(bld):
                                 'pugl_%s_stub_static' % platform],
                 uselib       = deps[platform]['uselib'] + ['CAIRO'])
 
+        # Make a hyper strict warning environment for checking API headers
+        strict_env = bld.env.derive()
+        autowaf.remove_all_warning_flags(strict_env)
+        autowaf.enable_all_warnings(strict_env)
+        autowaf.set_warnings_as_errors(strict_env)
+        autowaf.add_compiler_flags(strict_env, '*', {
+            'clang': ['-Wno-padded'],
+            'gcc': ['-Wno-padded'],
+        })
+        autowaf.add_compiler_flags(strict_env, 'cxx', {
+            'clang': ['-Wno-documentation-unknown-command'],
+        })
+
+        # Check that C headers build with (almost) no warnings
+        bld(features     = 'c cprogram',
+            source       = 'test/test_build.c',
+            target       = 'test/test_build_c',
+            install_path = '',
+            env          = strict_env,
+            use          = ['pugl_%s_static' % platform],
+            uselib       = deps[platform]['uselib'] + ['CAIRO'])
+
+        # Check that C++ headers build with (almost) no warnings
+        bld(features     = 'cxx cxxprogram',
+            source       = 'test/test_build.cpp',
+            target       = 'test/test_build_cpp',
+            install_path = '',
+            env          = strict_env,
+            use          = ['pugl_%s_static' % platform],
+            uselib       = deps[platform]['uselib'] + ['CAIRO'])
+
         if bld.env.CXX and bld.env.HAVE_GL:
             build_example('pugl_cxx_demo', ['examples/pugl_cxx_demo.cpp'],
                           platform, 'gl',