[PATCH] qt: fix warning -Wunused-local-typedefs introduced by gcc 4.8

* [PATCH] qt: fix warning -Wunused-local-typedefs introduced by gcc 4.8
@ 2013-05-16 23:28 Andreas Müller
  2013-05-20 16:14 ` Paul Eggleton
  0 siblings, 1 reply; 2+ messages in thread
From: Andreas Müller @ 2013-05-16 23:28 UTC (permalink / raw)
  To: openembedded-core

in meta-oe maliit-framework failed with

| In file included from /home/andreas/tmp/oe-core-eglibc/sysroots/overo/usr/include/qt4/QtCore/QtCore:53:0,
|                  from ../utils/core-utils.cpp:18:
| /home/andreas/tmp/oe-core-eglibc/sysroots/overo/usr/include/qt4/QtCore/qtconcurrentfilter.h: In function 'QtConcurrent::ThreadEngineStarter<void> QtConcurrent::filterInternal(Sequence&, KeepFunctor, Redu
| /home/andreas/tmp/oe-core-eglibc/sysroots/overo/usr/include/qt4/QtCore/qtconcurrentfilter.h:108:47: error: typedef 'Iterator' locally defined but not used [-Werror=unused-local-typedefs]
|      typedef typename Sequence::const_iterator Iterator;
|                                                ^
| cc1plus: all warnings being treated as errors

Signed-off-by: Andreas Müller <schnitzeltony@googlemail.com>
---
 meta/recipes-qt/qt4/qt4-4.8.4.inc                  |    1 +
 ...emove-unnecessary-typedef-found-by-gcc4-8.patch |   37 ++++++++++++++++++++
 meta/recipes-qt/qt4/qt4-embedded_4.8.4.bb          |    2 +-
 meta/recipes-qt/qt4/qt4-native.inc                 |    3 +-
 meta/recipes-qt/qt4/qt4-x11-free_4.8.4.bb          |    2 +-
 5 files changed, 42 insertions(+), 3 deletions(-)
 create mode 100644 meta/recipes-qt/qt4/qt4-4.8.4/0025-Remove-unnecessary-typedef-found-by-gcc4-8.patch

diff --git a/meta/recipes-qt/qt4/qt4-4.8.4.inc b/meta/recipes-qt/qt4/qt4-4.8.4.inc
index 3f0f003..965b617 100644
--- a/meta/recipes-qt/qt4/qt4-4.8.4.inc
+++ b/meta/recipes-qt/qt4/qt4-4.8.4.inc
@@ -24,6 +24,7 @@ SRC_URI = "http://releases.qt-project.org/qt4/source/qt-everywhere-opensource-sr
            file://0022-ssl-certificates-blacklist-mis-issued-turktrust-certificates.patch \
            file://0023-qtnetwork-blacklist-two-more-certificates.patch \
            file://0024-Change-all-shmget-calls-to-user-only-memory.patch \
+           file://0025-Remove-unnecessary-typedef-found-by-gcc4-8.patch \
            file://g++.conf \
            file://linux.conf \
            "
diff --git a/meta/recipes-qt/qt4/qt4-4.8.4/0025-Remove-unnecessary-typedef-found-by-gcc4-8.patch b/meta/recipes-qt/qt4/qt4-4.8.4/0025-Remove-unnecessary-typedef-found-by-gcc4-8.patch
new file mode 100644
index 0000000..498f14f
--- /dev/null
+++ b/meta/recipes-qt/qt4/qt4-4.8.4/0025-Remove-unnecessary-typedef-found-by-gcc4-8.patch
@@ -0,0 +1,37 @@
+From https://codereview.qt-project.org/p/qt/qt
+ * branch            refs/changes/22/53322/2 -> FETCH_HEAD
+From 28524b867ee8c9784f00eec0f07e4d9491a651b8 Mon Sep 17 00:00:00 2001
+From: Thiago Macieira <thiago.macieira@intel.com>
+Date: Tue, 9 Apr 2013 16:41:23 -0700
+Subject: [PATCH] Remove unnecessary typedef, found by GCC 4.8
+
+qtconcurrentfilter.h:108:47: warning: typedef 'Iterator' locally defined but not used [-Wunused-local-typedefs]
+
+Qt5 commit: d4c241a98059edaa7a4b15beb1cfd16ed041284e (qtbase)
+Task-number: QTBUG-30594
+Change-Id: Icc69e41c2d4e11962f0c8434236d4847b46e7d9d
+
+Upstream-Status: Pending for 4.8.5 [1]
+
+[1] https://codereview.qt-project.org/#change,53322
+
+Signed-off-by: Andreas Müller <schnitzeltony@googlemail.com>
+---
+ src/corelib/concurrent/qtconcurrentfilter.h |    1 -
+ 1 files changed, 0 insertions(+), 1 deletions(-)
+
+diff --git a/src/corelib/concurrent/qtconcurrentfilter.h b/src/corelib/concurrent/qtconcurrentfilter.h
+index 50b58bd..5a70182 100644
+--- a/src/corelib/concurrent/qtconcurrentfilter.h
++++ b/src/corelib/concurrent/qtconcurrentfilter.h
+@@ -105,7 +105,6 @@ namespace QtConcurrent {
+ template <typename Sequence, typename KeepFunctor, typename ReduceFunctor>
+ ThreadEngineStarter<void> filterInternal(Sequence &sequence, KeepFunctor keep, ReduceFunctor reduce)
+ {
+-    typedef typename Sequence::const_iterator Iterator;
+     typedef FilterKernel<Sequence, KeepFunctor, ReduceFunctor> KernelType;
+     return startThreadEngine(new KernelType(sequence, keep, reduce));
+ }
+-- 
+1.7.4.4
+
diff --git a/meta/recipes-qt/qt4/qt4-embedded_4.8.4.bb b/meta/recipes-qt/qt4/qt4-embedded_4.8.4.bb
index 286da95..c08927e 100644
--- a/meta/recipes-qt/qt4/qt4-embedded_4.8.4.bb
+++ b/meta/recipes-qt/qt4/qt4-embedded_4.8.4.bb
@@ -1,7 +1,7 @@
 require qt4-${PV}.inc
 require qt4-embedded.inc
 
-PR = "${INC_PR}.3"
+PR = "${INC_PR}.4"
 
 QT_CONFIG_FLAGS_append_arm = "${@bb.utils.contains("TUNE_FEATURES", "neon", "", " -no-neon" ,d)}"
 
diff --git a/meta/recipes-qt/qt4/qt4-native.inc b/meta/recipes-qt/qt4/qt4-native.inc
index 7645232..59c8ab5 100644
--- a/meta/recipes-qt/qt4/qt4-native.inc
+++ b/meta/recipes-qt/qt4/qt4-native.inc
@@ -9,7 +9,7 @@ LIC_FILES_CHKSUM = "file://LICENSE.LGPL;md5=fbc093901857fcd118f065f900982c24 \
                     file://LICENSE.GPL3;md5=6e1694ee338db410417517884918d4d2 \
                     file://LGPL_EXCEPTION.txt;md5=0145c4d1b6f96a661c2c139dfb268fb6"
 
-INC_PR = "r19"
+INC_PR = "r20"
 
 inherit native
 
@@ -18,6 +18,7 @@ SRC_URI = "http://releases.qt-project.org/qt4/source/qt-everywhere-opensource-sr
            file://0002-qkbdtty_qws-fix-build-with-old-kernel-headers.patch \
            file://0003-webkit2-set-OUTPUT_DIR-value-if-empty.patch \
            file://0021-configure-make-qt4-native-work-with-long-building-pa.patch \
+           file://0025-Remove-unnecessary-typedef-found-by-gcc4-8.patch \
            file://g++.conf \
            file://linux.conf \
 	"
diff --git a/meta/recipes-qt/qt4/qt4-x11-free_4.8.4.bb b/meta/recipes-qt/qt4/qt4-x11-free_4.8.4.bb
index e5db30d..d6edef9 100644
--- a/meta/recipes-qt/qt4/qt4-x11-free_4.8.4.bb
+++ b/meta/recipes-qt/qt4/qt4-x11-free_4.8.4.bb
@@ -1,7 +1,7 @@
 require qt4-x11-free.inc
 require qt4-${PV}.inc
 
-PR = "${INC_PR}.3"
+PR = "${INC_PR}.4"
 
 QT_CONFIG_FLAGS_append_arm = "${@bb.utils.contains("TUNE_FEATURES", "neon", "", " -no-neon" ,d)}"
 
-- 
1.7.4.4




^ permalink raw reply related	[flat|nested] 2+ messages in thread