From 3a29976eac1ff95e663d49886dc82611a175fed9 Mon Sep 17 00:00:00 2001 From: Marc Mutz Date: Mon, 27 Aug 2012 12:24:48 +0200 Subject: [PATCH] Move QtConcurrent::Exception as QException back to QtCore Leave the old name as a deprecated typedef; adapt users. This is a prerequisite for moving QFuture back to QtCore. Change-Id: I81dcee2c7e6eb234c16f3f42e2415ca0da3dc4f8 Reviewed-by: Thiago Macieira --- dist/changes-5.0.0 | 3 + src/concurrent/concurrent.pro | 1 - src/concurrent/qfutureinterface.cpp | 4 +- src/concurrent/qfutureinterface.h | 6 +- src/concurrent/qfutureinterface_p.h | 2 +- src/concurrent/qtconcurrentexception.h | 64 +---------- src/concurrent/qtconcurrentmap.cpp | 8 -- src/concurrent/qtconcurrentrunbase.h | 8 +- src/concurrent/qtconcurrentthreadengine.cpp | 10 +- src/concurrent/qtconcurrentthreadengine.h | 6 +- .../code/src_corelib_thread_qexception.cpp} | 4 +- .../thread/qexception.cpp} | 85 +++++++------- src/corelib/thread/qexception.h | 123 +++++++++++++++++++++ src/corelib/thread/thread.pri | 2 + tests/auto/concurrent/qfuture/tst_qfuture.cpp | 22 ++-- .../qtconcurrentmap/tst_qtconcurrentmap.cpp | 4 +- .../qtconcurrentrun/tst_qtconcurrentrun.cpp | 8 +- .../tst_qtconcurrentthreadengine.cpp | 14 +-- 18 files changed, 216 insertions(+), 158 deletions(-) rename src/{concurrent/doc/snippets/code/src_concurrent_qtconcurrentexception.cpp => corelib/doc/snippets/code/src_corelib_thread_qexception.cpp} (95%) rename src/{concurrent/qtconcurrentexception.cpp => corelib/thread/qexception.cpp} (67%) create mode 100644 src/corelib/thread/qexception.h diff --git a/dist/changes-5.0.0 b/dist/changes-5.0.0 index 5f2781c..b5f63c3 100644 --- a/dist/changes-5.0.0 +++ b/dist/changes-5.0.0 @@ -563,6 +563,9 @@ QtConcurrent QT += concurrent to include the new library. +* QtConcurrent::Exception has been renamed to QException, and is still in QtCore. + Ditto QtConcurrent::UnhandledException. + QtOpenGL -------- diff --git a/src/concurrent/concurrent.pro b/src/concurrent/concurrent.pro index 190da83..772146f 100644 --- a/src/concurrent/concurrent.pro +++ b/src/concurrent/concurrent.pro @@ -21,7 +21,6 @@ SOURCES += \ qtconcurrentresultstore.cpp \ qtconcurrentthreadengine.cpp \ qtconcurrentiteratekernel.cpp \ - qtconcurrentexception.cpp HEADERS += \ qtconcurrent_global.h \ diff --git a/src/concurrent/qfutureinterface.cpp b/src/concurrent/qfutureinterface.cpp index 5fc08c9..1f67552 100644 --- a/src/concurrent/qfutureinterface.cpp +++ b/src/concurrent/qfutureinterface.cpp @@ -238,7 +238,7 @@ void QFutureInterfaceBase::reportCanceled() } #ifndef QT_NO_EXCEPTIONS -void QFutureInterfaceBase::reportException(const QtConcurrent::Exception &exception) +void QFutureInterfaceBase::reportException(const QException &exception) { QMutexLocker locker(&d->m_mutex); if ((d->state & Canceled) || (d->state & Finished)) @@ -390,7 +390,7 @@ QMutex *QFutureInterfaceBase::mutex() const return &d->m_mutex; } -QtConcurrent::internal::ExceptionStore &QFutureInterfaceBase::exceptionStore() +QtPrivate::ExceptionStore &QFutureInterfaceBase::exceptionStore() { return d->m_exceptionStore; } diff --git a/src/concurrent/qfutureinterface.h b/src/concurrent/qfutureinterface.h index 55112ab..615b9fb 100644 --- a/src/concurrent/qfutureinterface.h +++ b/src/concurrent/qfutureinterface.h @@ -49,7 +49,7 @@ #ifndef QT_NO_QFUTURE #include -#include +#include #include QT_BEGIN_HEADER @@ -83,7 +83,7 @@ public: void reportFinished(); void reportCanceled(); #ifndef QT_NO_EXCEPTIONS - void reportException(const QtConcurrent::Exception &e); + void reportException(const QException &e); #endif void reportResultsReady(int beginIndex, int endIndex); @@ -122,7 +122,7 @@ public: void waitForResume(); QMutex *mutex() const; - QtConcurrent::internal::ExceptionStore &exceptionStore(); + QtPrivate::ExceptionStore &exceptionStore(); QtConcurrent::ResultStoreBase &resultStoreBase(); const QtConcurrent::ResultStoreBase &resultStoreBase() const; diff --git a/src/concurrent/qfutureinterface_p.h b/src/concurrent/qfutureinterface_p.h index e4c5d36..734584a 100644 --- a/src/concurrent/qfutureinterface_p.h +++ b/src/concurrent/qfutureinterface_p.h @@ -145,7 +145,7 @@ public: QtConcurrent::ResultStoreBase m_results; bool manualProgress; int m_expectedResultCount; - QtConcurrent::internal::ExceptionStore m_exceptionStore; + QtPrivate::ExceptionStore m_exceptionStore; QString m_progressText; QRunnable *runnable; diff --git a/src/concurrent/qtconcurrentexception.h b/src/concurrent/qtconcurrentexception.h index 0c86f08..373d42e 100644 --- a/src/concurrent/qtconcurrentexception.h +++ b/src/concurrent/qtconcurrentexception.h @@ -46,12 +46,7 @@ #ifndef QT_NO_QFUTURE -#include -#include - -#ifndef QT_NO_EXCEPTIONS -# include -#endif +#include QT_BEGIN_HEADER QT_BEGIN_NAMESPACE @@ -62,59 +57,8 @@ namespace QtConcurrent #ifndef QT_NO_EXCEPTIONS -class Q_CONCURRENT_EXPORT Exception : public std::exception -{ -public: - virtual void raise() const; - virtual Exception *clone() const; -}; - -class Q_CONCURRENT_EXPORT UnhandledException : public Exception -{ -public: - void raise() const; - Exception *clone() const; -}; - -namespace internal { - -class Base; -class ExceptionHolder -{ -public: - ExceptionHolder(Exception *exception = 0); - ExceptionHolder(const ExceptionHolder &other); - void operator=(const ExceptionHolder &other); - ~ExceptionHolder(); - Exception *exception() const; - QExplicitlySharedDataPointer base; -}; - -class Q_CONCURRENT_EXPORT ExceptionStore -{ -public: - void setException(const Exception &e); - bool hasException() const; - ExceptionHolder exception(); - void throwPossibleException(); - bool hasThrown() const; - ExceptionHolder exceptionHolder; -}; - -} // namespace internal - -#else // QT_NO_EXCEPTIONS - -namespace internal { - -class Q_CONCURRENT_EXPORT ExceptionStore -{ -public: - ExceptionStore() { } - inline void throwPossibleException() const {} -}; - -} // namespace internal +typedef Q_DECL_DEPRECATED QException Exception; +typedef Q_DECL_DEPRECATED QUnhandledException UnhandledException; #endif @@ -123,6 +67,6 @@ public: QT_END_NAMESPACE QT_END_HEADER -#endif // QT_NO_CONCURRENT +#endif // QT_NO_QFUTURE #endif diff --git a/src/concurrent/qtconcurrentmap.cpp b/src/concurrent/qtconcurrentmap.cpp index b26930c..f1e4617 100644 --- a/src/concurrent/qtconcurrentmap.cpp +++ b/src/concurrent/qtconcurrentmap.cpp @@ -55,14 +55,6 @@ */ /*! - \namespace QtConcurrent::internal - \internal - - \brief The QtConcurrent::internal namespace contains QtConcurrent - implementation details. -*/ - -/*! \enum QtConcurrent::ReduceOption This enum specifies the order of which results from the map or filter function are passed to the reduce function. diff --git a/src/concurrent/qtconcurrentrunbase.h b/src/concurrent/qtconcurrentrunbase.h index 02ce30a..897345f 100644 --- a/src/concurrent/qtconcurrentrunbase.h +++ b/src/concurrent/qtconcurrentrunbase.h @@ -104,10 +104,10 @@ public: #endif this->runFunctor(); #ifndef QT_NO_EXCEPTIONS - } catch (QtConcurrent::Exception &e) { + } catch (QException &e) { QFutureInterface::reportException(e); } catch (...) { - QFutureInterface::reportException(QtConcurrent::UnhandledException()); + QFutureInterface::reportException(QUnhandledException()); } #endif @@ -132,10 +132,10 @@ public: #endif this->runFunctor(); #ifndef QT_NO_EXCEPTIONS - } catch (QtConcurrent::Exception &e) { + } catch (QException &e) { QFutureInterface::reportException(e); } catch (...) { - QFutureInterface::reportException(QtConcurrent::UnhandledException()); + QFutureInterface::reportException(QUnhandledException()); } #endif this->reportFinished(); diff --git a/src/concurrent/qtconcurrentthreadengine.cpp b/src/concurrent/qtconcurrentthreadengine.cpp index 886fb09..5659296 100644 --- a/src/concurrent/qtconcurrentthreadengine.cpp +++ b/src/concurrent/qtconcurrentthreadengine.cpp @@ -155,10 +155,10 @@ void ThreadEngineBase::startBlocking() } } #ifndef QT_NO_EXCEPTIONS - } catch (QtConcurrent::Exception &e) { + } catch (QException &e) { handleException(e); } catch (...) { - handleException(QtConcurrent::UnhandledException()); + handleException(QUnhandledException()); } #endif @@ -271,10 +271,10 @@ void ThreadEngineBase::run() // implements QRunnable. } #ifndef QT_NO_EXCEPTIONS - } catch (QtConcurrent::Exception &e) { + } catch (QException &e) { handleException(e); } catch (...) { - handleException(QtConcurrent::UnhandledException()); + handleException(QUnhandledException()); } #endif threadExit(); @@ -282,7 +282,7 @@ void ThreadEngineBase::run() // implements QRunnable. #ifndef QT_NO_EXCEPTIONS -void ThreadEngineBase::handleException(const QtConcurrent::Exception &exception) +void ThreadEngineBase::handleException(const QException &exception) { if (futureInterface) futureInterface->reportException(exception); diff --git a/src/concurrent/qtconcurrentthreadengine.h b/src/concurrent/qtconcurrentthreadengine.h index 971348b..2a51875 100644 --- a/src/concurrent/qtconcurrentthreadengine.h +++ b/src/concurrent/qtconcurrentthreadengine.h @@ -49,7 +49,7 @@ #include #include #include -#include +#include #include #include #include @@ -119,13 +119,13 @@ private: void run(); virtual void asynchronousFinish() = 0; #ifndef QT_NO_EXCEPTIONS - void handleException(const QtConcurrent::Exception &exception); + void handleException(const QException &exception); #endif protected: QFutureInterfaceBase *futureInterface; QThreadPool *threadPool; ThreadEngineBarrier barrier; - QtConcurrent::internal::ExceptionStore exceptionStore; + QtPrivate::ExceptionStore exceptionStore; }; diff --git a/src/concurrent/doc/snippets/code/src_concurrent_qtconcurrentexception.cpp b/src/corelib/doc/snippets/code/src_corelib_thread_qexception.cpp similarity index 95% rename from src/concurrent/doc/snippets/code/src_concurrent_qtconcurrentexception.cpp rename to src/corelib/doc/snippets/code/src_corelib_thread_qexception.cpp index 002d669..d2bc636 100644 --- a/src/concurrent/doc/snippets/code/src_concurrent_qtconcurrentexception.cpp +++ b/src/corelib/doc/snippets/code/src_corelib_thread_qexception.cpp @@ -40,11 +40,11 @@ //! [0] -class MyException : public QtConcurrent::Exception +class MyException : public QException { public: void raise() const { throw *this; } - Exception *clone() const { return new MyException(*this); } + MyException *clone() const { return new MyException(*this); } }; //! [0] diff --git a/src/concurrent/qtconcurrentexception.cpp b/src/corelib/thread/qexception.cpp similarity index 67% rename from src/concurrent/qtconcurrentexception.cpp rename to src/corelib/thread/qexception.cpp index 82fa5ab..090cbf4 100644 --- a/src/concurrent/qtconcurrentexception.cpp +++ b/src/corelib/thread/qexception.cpp @@ -39,7 +39,7 @@ ** ****************************************************************************/ -#include "qtconcurrentexception.h" +#include "qexception.h" #include "QtCore/qshareddata.h" #ifndef QT_NO_QFUTURE @@ -48,23 +48,23 @@ QT_BEGIN_NAMESPACE /*! - \class QtConcurrent::Exception - \brief The Exception class provides a base class for exceptions that can transferred across threads. - \since 4.4 + \class QException + \brief The QException class provides a base class for exceptions that can transferred across threads. + \since 5.0 Qt Concurrent supports throwing and catching exceptions across thread - boundaries, provided that the exception inherit from QtConcurrent::Exception + boundaries, provided that the exception inherit from QException and implement two helper functions: - \snippet code/src_concurrent_qtconcurrentexception.cpp 0 + \snippet code/src_corelib_thread_qexception.cpp 0 - QtConcurrent::Exception subclasses must be thrown by value and + QException subclasses must be thrown by value and caught by reference: - \snippet code/src_concurrent_qtconcurrentexception.cpp 1 + \snippet code/src_corelib_thread_qexception.cpp 1 - If you throw an exception that is not a subclass of QtConcurrent::Exception, - the Qt Concurrent functions will throw a QtConcurrent::UnhandledException + If you throw an exception that is not a subclass of QException, + the Qt functions will throw a QUnhandledException in the receiver thread. When using QFuture, transferred exceptions will be thrown when calling the following functions: @@ -77,83 +77,80 @@ QT_BEGIN_NAMESPACE */ /*! - \fn QtConcurrent::Exception::raise() const - In your QtConcurrent::Exception subclass, reimplement raise() like this: - - \snippet code/src_concurrent_qtconcurrentexception.cpp 2 + \fn QException::raise() const + In your QException subclass, reimplement raise() like this: + + \snippet code/src_corelib_thread_qexception.cpp 2 */ /*! - \fn QtConcurrent::Exception::clone() const - In your QtConcurrent::Exception subclass, reimplement clone() like this: - - \snippet code/src_concurrent_qtconcurrentexception.cpp 3 + \fn QException::clone() const + In your QException subclass, reimplement clone() like this: + + \snippet code/src_corelib_thread_qexception.cpp 3 */ /*! - \class QtConcurrent::UnhandledException + \class QUnhandledException \brief The UnhandledException class represents an unhandled exception in a worker thread. - \since 4.4 + \since 5.0 - If a worker thread throws an exception that is not a subclass of QtConcurrent::Exception, - the Qt Concurrent functions will throw a QtConcurrent::UnhandledException + If a worker thread throws an exception that is not a subclass of QException, + the Qt functions will throw a QUnhandledException on the receiver thread side. Inheriting from this class is not supported. */ /*! - \fn QtConcurrent::UnhandledException::raise() const + \fn QUnhandledException::raise() const \internal */ /*! - \fn QtConcurrent::UnhandledException::clone() const + \fn QUnhandledException::clone() const \internal */ -namespace QtConcurrent +void QException::raise() const { - -void Exception::raise() const -{ - Exception e = *this; + QException e = *this; throw e; } -Exception *Exception::clone() const +QException *QException::clone() const { - return new Exception(*this); + return new QException(*this); } -void UnhandledException::raise() const +void QUnhandledException::raise() const { - UnhandledException e = *this; + QUnhandledException e = *this; throw e; } -Exception *UnhandledException::clone() const +QUnhandledException *QUnhandledException::clone() const { - return new UnhandledException(*this); + return new QUnhandledException(*this); } #ifndef qdoc -namespace internal { +namespace QtPrivate { class Base : public QSharedData { public: - Base(Exception *exception) + Base(QException *exception) : exception(exception), hasThrown(false) { } ~Base() { delete exception; } - Exception *exception; + QException *exception; bool hasThrown; }; -ExceptionHolder::ExceptionHolder(Exception *exception) +ExceptionHolder::ExceptionHolder(QException *exception) : base(new Base(exception)) {} ExceptionHolder::ExceptionHolder(const ExceptionHolder &other) @@ -168,12 +165,12 @@ void ExceptionHolder::operator=(const ExceptionHolder &other) ExceptionHolder::~ExceptionHolder() {} -Exception *ExceptionHolder::exception() const +QException *ExceptionHolder::exception() const { return base->exception; } -void ExceptionStore::setException(const Exception &e) +void ExceptionStore::setException(const QException &e) { if (hasException() == false) exceptionHolder = ExceptionHolder(e.clone()); @@ -199,13 +196,11 @@ void ExceptionStore::throwPossibleException() bool ExceptionStore::hasThrown() const { return exceptionHolder.base->hasThrown; } -} // namespace internal +} // namespace QtPrivate #endif //qdoc -} // namespace QtConcurrent - QT_END_NAMESPACE #endif // QT_NO_EXCEPTIONS -#endif // QT_NO_CONCURRENT +#endif // QT_NO_QFUTURE diff --git a/src/corelib/thread/qexception.h b/src/corelib/thread/qexception.h new file mode 100644 index 0000000..7106bb5 --- /dev/null +++ b/src/corelib/thread/qexception.h @@ -0,0 +1,123 @@ +/**************************************************************************** +** +** Copyright (C) 2012 Digia Plc and/or its subsidiary(-ies). +** Contact: http://www.qt-project.org/legal +** +** This file is part of the QtCore module of the Qt Toolkit. +** +** $QT_BEGIN_LICENSE:LGPL$ +** Commercial License Usage +** Licensees holding valid commercial Qt licenses may use this file in +** accordance with the commercial license agreement provided with the +** Software or, alternatively, in accordance with the terms contained in +** a written agreement between you and Digia. For licensing terms and +** conditions see http://qt.digia.com/licensing. For further information +** use the contact form at http://qt.digia.com/contact-us. +** +** GNU Lesser General Public License Usage +** Alternatively, this file may be used under the terms of the GNU Lesser +** General Public License version 2.1 as published by the Free Software +** Foundation and appearing in the file LICENSE.LGPL included in the +** packaging of this file. Please review the following information to +** ensure the GNU Lesser General Public License version 2.1 requirements +** will be met: http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html. +** +** In addition, as a special exception, Digia gives you certain additional +** rights. These rights are described in the Digia Qt LGPL Exception +** version 1.1, included in the file LGPL_EXCEPTION.txt in this package. +** +** GNU General Public License Usage +** Alternatively, this file may be used under the terms of the GNU +** General Public License version 3.0 as published by the Free Software +** Foundation and appearing in the file LICENSE.GPL included in the +** packaging of this file. Please review the following information to +** ensure the GNU General Public License version 3.0 requirements will be +** met: http://www.gnu.org/copyleft/gpl.html. +** +** +** $QT_END_LICENSE$ +** +****************************************************************************/ + +#ifndef QTCORE_QEXCEPTION_H +#define QTCORE_QEXCEPTION_H + +#include + +#ifndef QT_NO_QFUTURE + +#include +#include + +#ifndef QT_NO_EXCEPTIONS +# include +#endif + +QT_BEGIN_HEADER +QT_BEGIN_NAMESPACE + + +#ifndef QT_NO_EXCEPTIONS + +class Q_CORE_EXPORT QException : public std::exception +{ +public: + virtual void raise() const; + virtual QException *clone() const; +}; + +class Q_CORE_EXPORT QUnhandledException : public QException +{ +public: + void raise() const; + QUnhandledException *clone() const; +}; + +namespace QtPrivate { + +class Base; +class Q_CORE_EXPORT ExceptionHolder +{ +public: + ExceptionHolder(QException *exception = 0); + ExceptionHolder(const ExceptionHolder &other); + void operator=(const ExceptionHolder &other); + ~ExceptionHolder(); + QException *exception() const; + QExplicitlySharedDataPointer base; +}; + +class Q_CORE_EXPORT ExceptionStore +{ +public: + void setException(const QException &e); + bool hasException() const; + ExceptionHolder exception(); + void throwPossibleException(); + bool hasThrown() const; + ExceptionHolder exceptionHolder; +}; + +} // namespace QtPrivate + +#else // QT_NO_EXCEPTIONS + +namespace QtPrivate { + +class Q_CORE_EXPORT ExceptionStore +{ +public: + ExceptionStore() { } + inline void throwPossibleException() const {} +}; + +} // namespace QtPrivate + +#endif // QT_NO_EXCEPTIONS + +QT_END_NAMESPACE +QT_END_HEADER + +#endif // QT_NO_QFUTURE + +#endif diff --git a/src/corelib/thread/thread.pri b/src/corelib/thread/thread.pri index 7f383d6..215e566 100644 --- a/src/corelib/thread/thread.pri +++ b/src/corelib/thread/thread.pri @@ -10,6 +10,7 @@ HEADERS += thread/qmutex.h \ thread/qthreadstorage.h \ thread/qwaitcondition.h \ thread/qatomic.h \ + thread/qexception.h \ thread/qbasicatomic.h \ thread/qgenericatomic.h \ thread/qoldbasicatomic.h @@ -23,6 +24,7 @@ HEADERS += thread/qmutex_p.h \ thread/qthreadpool_p.h SOURCES += thread/qatomic.cpp \ + thread/qexception.cpp \ thread/qmutex.cpp \ thread/qreadwritelock.cpp \ thread/qrunnable.cpp \ diff --git a/tests/auto/concurrent/qfuture/tst_qfuture.cpp b/tests/auto/concurrent/qfuture/tst_qfuture.cpp index 4e681e7..aea35e8 100644 --- a/tests/auto/concurrent/qfuture/tst_qfuture.cpp +++ b/tests/auto/concurrent/qfuture/tst_qfuture.cpp @@ -1276,7 +1276,7 @@ QFuture createExceptionFuture() i.reportStarted(); QFuture f = i.future(); - Exception e; + QException e; i.reportException(e); i.reportFinished(); return f; @@ -1290,17 +1290,17 @@ QFuture createExceptionResultFuture() int r = 0; i.reportResult(r); - Exception e; + QException e; i.reportException(e); i.reportFinished(); return f; } -class DerivedException : public Exception +class DerivedException : public QException { public: void raise() const { throw *this; } - Exception *clone() const { return new DerivedException(*this); } + DerivedException *clone() const { return new DerivedException(*this); } }; QFuture createDerivedExceptionFuture() @@ -1323,7 +1323,7 @@ void tst_QFuture::exceptions() bool caught = false; try { f.waitForFinished(); - } catch (Exception &) { + } catch (QException &) { caught = true; } QVERIFY(caught); @@ -1335,7 +1335,7 @@ void tst_QFuture::exceptions() bool caught = false; try { f.result(); - } catch (Exception &) { + } catch (QException &) { caught = true; } QVERIFY(caught); @@ -1346,7 +1346,7 @@ void tst_QFuture::exceptions() bool caught = false; try { createExceptionResultFuture().result(); - } catch (Exception &) { + } catch (QException &) { caught = true; } QVERIFY(caught); @@ -1358,7 +1358,7 @@ void tst_QFuture::exceptions() bool caught = false; try { f.results(); - } catch (Exception &) { + } catch (QException &) { caught = true; } QVERIFY(caught); @@ -1373,7 +1373,7 @@ void tst_QFuture::exceptions() Q_UNUSED(e); QFAIL("did not get exception"); } - } catch (Exception &) { + } catch (QException &) { caught = true; } QVERIFY(caught); @@ -1384,7 +1384,7 @@ void tst_QFuture::exceptions() bool caught = false; try { createDerivedExceptionFuture().waitForFinished(); - } catch (Exception &) { + } catch (QException &) { caught = true; } QVERIFY(caught); @@ -1409,7 +1409,7 @@ public: QFuture f = createExceptionFuture(); try { f.waitForFinished(); - } catch (Exception &) { + } catch (QException &) { caught = true; } } diff --git a/tests/auto/concurrent/qtconcurrentmap/tst_qtconcurrentmap.cpp b/tests/auto/concurrent/qtconcurrentmap/tst_qtconcurrentmap.cpp index 48837d3..186ee82 100644 --- a/tests/auto/concurrent/qtconcurrentmap/tst_qtconcurrentmap.cpp +++ b/tests/auto/concurrent/qtconcurrentmap/tst_qtconcurrentmap.cpp @@ -2205,7 +2205,7 @@ void tst_QtConcurrentMap::throttling() void throwMapper(int &e) { Q_UNUSED(e); - throw QtConcurrent::Exception(); + throw QException(); } void tst_QtConcurrentMap::exceptions() @@ -2214,7 +2214,7 @@ void tst_QtConcurrentMap::exceptions() try { QList list = QList() << 1 << 2 << 3; QtConcurrent::map(list, throwMapper).waitForFinished(); - } catch (const Exception &) { + } catch (const QException &) { caught = true; } if (!caught) diff --git a/tests/auto/concurrent/qtconcurrentrun/tst_qtconcurrentrun.cpp b/tests/auto/concurrent/qtconcurrentrun/tst_qtconcurrentrun.cpp index af63c25..f3f4be3 100644 --- a/tests/auto/concurrent/qtconcurrentrun/tst_qtconcurrentrun.cpp +++ b/tests/auto/concurrent/qtconcurrentrun/tst_qtconcurrentrun.cpp @@ -367,12 +367,12 @@ int fn2(double, int *) #ifndef QT_NO_EXCEPTIONS void throwFunction() { - throw QtConcurrent::Exception(); + throw QException(); } int throwFunctionReturn() { - throw QtConcurrent::Exception(); + throw QException(); return 0; } @@ -381,7 +381,7 @@ void tst_QtConcurrentRun::exceptions() bool caught = false; try { QtConcurrent::run(throwFunction).waitForFinished(); - } catch (Exception &) { + } catch (QException &) { caught = true; } if (!caught) @@ -390,7 +390,7 @@ void tst_QtConcurrentRun::exceptions() caught = false; try { QtConcurrent::run(throwFunctionReturn).waitForFinished(); - } catch (Exception &) { + } catch (QException &) { caught = true; } if (!caught) diff --git a/tests/auto/concurrent/qtconcurrentthreadengine/tst_qtconcurrentthreadengine.cpp b/tests/auto/concurrent/qtconcurrentthreadengine/tst_qtconcurrentthreadengine.cpp index d2e4c74..edf9104 100644 --- a/tests/auto/concurrent/qtconcurrentthreadengine/tst_qtconcurrentthreadengine.cpp +++ b/tests/auto/concurrent/qtconcurrentthreadengine/tst_qtconcurrentthreadengine.cpp @@ -425,7 +425,7 @@ public: ThreadFunctionResult threadFunction() { QTest::qSleep(50); - throw QtConcurrent::Exception(); + throw QException(); return ThreadFinished; } QThread *blockThread; @@ -457,7 +457,7 @@ void tst_QtConcurrentThreadEngine::exceptions() QtConcurrentExceptionThrower *e = new QtConcurrentExceptionThrower(); QFuture f = e->startAsynchronously(); f.waitForFinished(); - } catch (const Exception &) { + } catch (const QException &) { caught = true; } QVERIFY2(caught, "did not get exception"); @@ -470,7 +470,7 @@ void tst_QtConcurrentThreadEngine::exceptions() try { QtConcurrentExceptionThrower e(QThread::currentThread()); e.startBlocking(); - } catch (const Exception &) { + } catch (const QException &) { caught = true; } QVERIFY2(caught, "did not get exception"); @@ -482,7 +482,7 @@ void tst_QtConcurrentThreadEngine::exceptions() try { QtConcurrentExceptionThrower e(0); e.startBlocking(); - } catch (const Exception &) { + } catch (const QException &) { caught = true; } QVERIFY2(caught, "did not get exception"); @@ -495,7 +495,7 @@ void tst_QtConcurrentThreadEngine::exceptions() UnrelatedExceptionThrower *e = new UnrelatedExceptionThrower(); QFuture f = e->startAsynchronously(); f.waitForFinished(); - } catch (const QtConcurrent::UnhandledException &) { + } catch (const QUnhandledException &) { caught = true; } QVERIFY2(caught, "did not get exception"); @@ -508,7 +508,7 @@ void tst_QtConcurrentThreadEngine::exceptions() try { UnrelatedExceptionThrower e(QThread::currentThread()); e.startBlocking(); - } catch (const QtConcurrent::UnhandledException &) { + } catch (const QUnhandledException &) { caught = true; } QVERIFY2(caught, "did not get exception"); @@ -520,7 +520,7 @@ void tst_QtConcurrentThreadEngine::exceptions() try { UnrelatedExceptionThrower e(0); e.startBlocking(); - } catch (const QtConcurrent::UnhandledException &) { + } catch (const QUnhandledException &) { caught = true; } QVERIFY2(caught, "did not get exception"); -- 2.7.4