concurrent: Fix missing or improper include guard in headers
authorSergio Ahumada <sergio.ahumada@nokia.com>
Sat, 15 Sep 2012 15:31:44 +0000 (17:31 +0200)
committerQt by Nokia <qt-info@nokia.com>
Sat, 15 Sep 2012 21:54:29 +0000 (23:54 +0200)
Use an include guard in headers to ensure the header is not included
more than once. Make the header guard match its file name.

Also, cpp files should include their own headers first (but below config.h)

Change-Id: I902c8936382f5c1a8e0de7dbf49e5423f9b72bbe
Reviewed-by: Lars Knoll <lars.knoll@nokia.com>
src/concurrent/qfutureinterface.cpp
src/concurrent/qfuturesynchronizer.h
src/concurrent/qfuturewatcher.cpp

index 9a273a1..0836ad0 100644 (file)
 
 #ifndef QT_NO_QFUTURE
 
+#include "qfutureinterface_p.h"
+
 #include <QtCore/qatomic.h>
 #include <QtCore/qthread.h>
 #include <QtCore/qthreadpool.h>
 #include <private/qthreadpool_p.h>
 
-#include "qfutureinterface_p.h"
-
 QT_BEGIN_NAMESPACE
 
 enum {
index 3106dd4..36208d7 100644 (file)
@@ -39,8 +39,8 @@
 **
 ****************************************************************************/
 
-#ifndef QFUTRUESYNCHRONIZER_H
-#define QFUTRUESYNCHRONIZER_H
+#ifndef QFUTURESYNCHRONIZER_H
+#define QFUTURESYNCHRONIZER_H
 
 #include <QtConcurrent/qtconcurrent_global.h>
 
@@ -119,4 +119,4 @@ QT_END_HEADER
 
 #endif // QT_NO_CONCURRENT
 
-#endif // QFUTRUESYNCHRONIZER_H
+#endif // QFUTURESYNCHRONIZER_H
index eb5fb0e..081b21c 100644 (file)
 
 #ifndef QT_NO_QFUTURE
 
+#include "qfuturewatcher_p.h"
+
 #include <QtCore/qcoreevent.h>
 #include <QtCore/qcoreapplication.h>
 #include <QtCore/qmetaobject.h>
 #include <QtCore/qthread.h>
 
-#include "qfuturewatcher_p.h"
-
 QT_BEGIN_NAMESPACE
 
 /*! \class QFutureWatcher