These issues wont happen with qt3d master since assimp seems to have dropped the offending code, but for the release we use in meta-qt5 as of now we will need it.
Signed-off-by: Khem Raj <[email protected]> --- ...001-Fix-narrowing-errors-found-with-clang.patch | 72 ++++++++++++++++++++++ recipes-qt/qt5/qt3d_git.bb | 2 + 2 files changed, 74 insertions(+) create mode 100644 recipes-qt/qt5/qt3d/0001-Fix-narrowing-errors-found-with-clang.patch diff --git a/recipes-qt/qt5/qt3d/0001-Fix-narrowing-errors-found-with-clang.patch b/recipes-qt/qt5/qt3d/0001-Fix-narrowing-errors-found-with-clang.patch new file mode 100644 index 0000000..afe7d32 --- /dev/null +++ b/recipes-qt/qt5/qt3d/0001-Fix-narrowing-errors-found-with-clang.patch @@ -0,0 +1,72 @@ +From 538f94f658b181c95c4e98be1429f798be1b437f Mon Sep 17 00:00:00 2001 +From: Khem Raj <[email protected]> +Date: Sun, 23 Aug 2015 15:50:38 -0700 +Subject: [PATCH] Fix narrowing errors found with clang + +clang errors out with + +non-constant-expression cannot be narrowed from type 'int8_t' (aka +'signed char') to 'char' in initializer list + +adjust the types to fix the error + +string contructor expects char* and not unsigned char* so adjust the +call + +Another issue is seen in constructor using bool instead of a pointer + +error: cannot initialize a + member subobject of type 'Assimp::DXFImporter::LayerInfo *' with + an rvalue of type 'bool' + +Signed-off-by: Khem Raj <[email protected]> +--- + 3rdparty/assimp/code/BlenderDNA.cpp | 6 +++--- + 3rdparty/assimp/code/DXFLoader.cpp | 2 +- + 2 files changed, 4 insertions(+), 4 deletions(-) + +diff --git a/3rdparty/assimp/code/BlenderDNA.cpp b/3rdparty/assimp/code/BlenderDNA.cpp +index 75ef61f..fb6d1de 100644 +--- a/3rdparty/assimp/code/BlenderDNA.cpp ++++ b/3rdparty/assimp/code/BlenderDNA.cpp +@@ -55,7 +55,7 @@ using namespace Assimp::Formatter; + + #define for_each BOOST_FOREACH + bool match4(StreamReaderAny& stream, const char* string) { +- char tmp[] = { ++ int8_t tmp[] = { + (stream).GetI1(), + (stream).GetI1(), + (stream).GetI1(), +@@ -343,13 +343,13 @@ void SectionParser :: Next() + { + stream.SetCurrentPos(current.start + current.size); + +- const char tmp[] = { ++ const int8_t tmp[] = { + stream.GetI1(), + stream.GetI1(), + stream.GetI1(), + stream.GetI1() + }; +- current.id = std::string(tmp,tmp[3]?4:tmp[2]?3:tmp[1]?2:1); ++ current.id = std::string((char*)tmp,tmp[3]?4:tmp[2]?3:tmp[1]?2:1); + + current.size = stream.GetI4(); + current.address.val = ptr64 ? stream.GetU8() : stream.GetU4(); +diff --git a/3rdparty/assimp/code/DXFLoader.cpp b/3rdparty/assimp/code/DXFLoader.cpp +index e372d60..159e1a9 100644 +--- a/3rdparty/assimp/code/DXFLoader.cpp ++++ b/3rdparty/assimp/code/DXFLoader.cpp +@@ -84,7 +84,7 @@ aiColor4D g_clrInvalid = aiColor4D(get_qnan(),0.f,0.f,1.f); + + // ------------------------------------------------------------------------------------------------ + // Constructor to be privately used by Importer +-DXFImporter::DXFImporter() : buffer(0), groupCode(0), bRepeat(false), mDefaultLayer(false) ++DXFImporter::DXFImporter() : buffer(0), groupCode(0), bRepeat(false), mDefaultLayer(NULL) + { + memset(cursor,0,sizeof(cursor)); + } +-- +2.1.4 + diff --git a/recipes-qt/qt5/qt3d_git.bb b/recipes-qt/qt5/qt3d_git.bb index 62f17e1..752d896 100644 --- a/recipes-qt/qt5/qt3d_git.bb +++ b/recipes-qt/qt5/qt3d_git.bb @@ -21,3 +21,5 @@ QT_MODULE_BRANCH = "dev" # last tag before this SRCREV is 5.0.0-beta1, but version says 5.3.0 already SRCREV = "8a9723d7420829a2ba0141a9f7d8dc8cd11d8331" + +SRC_URI += "file://0001-Fix-narrowing-errors-found-with-clang.patch" -- 2.1.4 -- _______________________________________________ Openembedded-devel mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-devel
