Clean up the headers

They MUST compile in pedantic, -Werror=shadow, no-keywords mode. In addition
to that, this fixes:

error: typedef ‘qHandle’ locally defined but not used [-Werror=unused-local-typedefs]

Change-Id: If39e049dee0d2a468fcd524b18597e3c088c0bc3
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Fix warning about unused parameters

openalaspect.cpp:65:6: warning: unused parameter ‘rootObject’ [-Wunused-parameter]

Change-Id: I95b76551c3c997d3ab394b1284e3d52293604379
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Fix compilation: ISO C++ does not allow anonymous structs

qhandle.h:103:9: error: ISO C++ prohibits anonymous structs [-Wpedantic]

Change-Id: I06e6736d96ad24968d276e721ce53752ee99ebeb
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Add virtual destructors to classes containing virtuals

Change-Id: I24b8c0524a98b5415be997344db684ea3f81a9e4
error: ‘class Qt3D::Render::RenderState’ has virtual functions and accessible non-virtual destructor
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Remove the << version of qCWarning from headers

This triggers warnings in pedantic compilation mode since passing zero
arguments to the ... is not permitted.

GCC:   quniformvalue.h:188:38: error: ISO C99 requires rest arguments to be used
Clang: quniformvalue.h:141:38: error: must specify at least one argument for '...' parameter of variadic macro [-Werror,-Wgnu-zero-variadic-macro-arguments]

This commit also removes the Q_FUNC_INFO, which isn't necessary. If the
user wants it, they can set QT_MESSAGE_PATTERN in the environment.

Discussed-on: http://lists.qt-project.org/pipermail/development/2014-August/018022.html
Change-Id: Ic75c669f2b5364033d8987f130b5d24a5cf80454
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
ICC doesn't have -Wno-ignored-qualifiers

It prints an annoying warning for every .cpp
icpc: command line warning #10006: ignoring unknown option '-Wno-ignored-qualifiers'

It has its own option for the same warning. Let's use it.

Change-Id: Ia540dc8836873d8386f8db092a29af41530c7c21
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Ignore the ICC warning about K&R-style parameters in C

It's 2014. There are still people using this?

Change-Id: Ie53cd86e9794a98919579fe1dbe1502e7e9ea3d9
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Fix use of variable before initialization

Caught by ICC.
qcuboidmesh.cpp(75): warning #592: variable "mesh" is used before its value is set

Change-Id: I9f1a82e17c0d22c1c2b2ec0ca0b953f9ecb16644
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
shaderpropertyparser is not a public header

It includes private QtQml headers. Therefore, it's also private.

Change-Id: Ic717e04e762193abdf4da2398269d75d8fc240c8
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>