diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2012-08-27 12:24:48 +0200 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2012-09-28 23:50:10 +0200 |
commit | 3a29976eac1ff95e663d49886dc82611a175fed9 (patch) | |
tree | 31d70d25501a0f95e17b65e4c029c5c40b75610f /src | |
parent | 5414a9a03bec65fd180f3c29811397df14130881 (diff) |
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 <thiago.macieira@intel.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/concurrent/concurrent.pro | 1 | ||||
-rw-r--r-- | src/concurrent/qfutureinterface.cpp | 4 | ||||
-rw-r--r-- | src/concurrent/qfutureinterface.h | 6 | ||||
-rw-r--r-- | src/concurrent/qfutureinterface_p.h | 2 | ||||
-rw-r--r-- | src/concurrent/qtconcurrentexception.h | 64 | ||||
-rw-r--r-- | src/concurrent/qtconcurrentmap.cpp | 8 | ||||
-rw-r--r-- | src/concurrent/qtconcurrentrunbase.h | 8 | ||||
-rw-r--r-- | src/concurrent/qtconcurrentthreadengine.cpp | 10 | ||||
-rw-r--r-- | src/concurrent/qtconcurrentthreadengine.h | 6 | ||||
-rw-r--r-- | src/corelib/doc/snippets/code/src_corelib_thread_qexception.cpp (renamed from src/concurrent/doc/snippets/code/src_concurrent_qtconcurrentexception.cpp) | 4 | ||||
-rw-r--r-- | src/corelib/thread/qexception.cpp (renamed from src/concurrent/qtconcurrentexception.cpp) | 85 | ||||
-rw-r--r-- | src/corelib/thread/qexception.h | 123 | ||||
-rw-r--r-- | src/corelib/thread/thread.pri | 2 |
13 files changed, 189 insertions, 134 deletions
diff --git a/src/concurrent/concurrent.pro b/src/concurrent/concurrent.pro index 190da83e9c..772146f682 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 5fc08c9366..1f675525a5 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 55112ab68d..615b9fbe6a 100644 --- a/src/concurrent/qfutureinterface.h +++ b/src/concurrent/qfutureinterface.h @@ -49,7 +49,7 @@ #ifndef QT_NO_QFUTURE #include <QtCore/qmutex.h> -#include <QtConcurrent/qtconcurrentexception.h> +#include <QtCore/qexception.h> #include <QtConcurrent/qtconcurrentresultstore.h> 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 e4c5d369aa..734584adad 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 0c86f08d75..373d42ee64 100644 --- a/src/concurrent/qtconcurrentexception.h +++ b/src/concurrent/qtconcurrentexception.h @@ -46,12 +46,7 @@ #ifndef QT_NO_QFUTURE -#include <QtCore/qatomic.h> -#include <QtCore/qshareddata.h> - -#ifndef QT_NO_EXCEPTIONS -# include <exception> -#endif +#include <QtCore/qexception.h> 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> 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 b26930cf0c..f1e46177eb 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 02ce30a7f5..897345f239 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<T>::reportException(e); } catch (...) { - QFutureInterface<T>::reportException(QtConcurrent::UnhandledException()); + QFutureInterface<T>::reportException(QUnhandledException()); } #endif @@ -132,10 +132,10 @@ public: #endif this->runFunctor(); #ifndef QT_NO_EXCEPTIONS - } catch (QtConcurrent::Exception &e) { + } catch (QException &e) { QFutureInterface<void>::reportException(e); } catch (...) { - QFutureInterface<void>::reportException(QtConcurrent::UnhandledException()); + QFutureInterface<void>::reportException(QUnhandledException()); } #endif this->reportFinished(); diff --git a/src/concurrent/qtconcurrentthreadengine.cpp b/src/concurrent/qtconcurrentthreadengine.cpp index 886fb09ea6..56592965fd 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 971348be97..2a5187502f 100644 --- a/src/concurrent/qtconcurrentthreadengine.h +++ b/src/concurrent/qtconcurrentthreadengine.h @@ -49,7 +49,7 @@ #include <QtCore/qthreadpool.h> #include <QtConcurrent/qfuture.h> #include <QtCore/qdebug.h> -#include <QtConcurrent/qtconcurrentexception.h> +#include <QtCore/qexception.h> #include <QtCore/qwaitcondition.h> #include <QtCore/qatomic.h> #include <QtCore/qsemaphore.h> @@ -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 index 002d669493..d2bc6363ed 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 index 82fa5ab243..090cbf4e85 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 0000000000..7106bb57df --- /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 <QtCore/qglobal.h> + +#ifndef QT_NO_QFUTURE + +#include <QtCore/qatomic.h> +#include <QtCore/qshareddata.h> + +#ifndef QT_NO_EXCEPTIONS +# include <exception> +#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> 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 7f383d6a4c..215e56628c 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 \ |