From 80a431a2e6369b471e07905be1b930669a4aae27 Mon Sep 17 00:00:00 2001 From: Thiago Macieira Date: Fri, 7 Jul 2023 09:20:57 -0700 Subject: [PATCH] QBasicAtomicXXX: remove the comment about them being "New atomics" MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit They were new... in commit 1f843ca39ee59c5304009faf308ddce791614a02 ("Add the new QBasicAtomicXXX implementation - no backends yet") for Qt 5.0, which is what allowed us to use the C++11 atomics. That replaced the even older implementation from Qt 4.4 where QBasicAtomicInt was a full class and we didn't support any other integer types. That of course replaced the even older QBasicAtomic from 4.0, first introduced in 2004 (7 years before C++11). Change-Id: I53335f845a1345299031fffd176fa251b31bfeea Reviewed-by: Mårten Nordheim --- src/corelib/thread/qbasicatomic.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/corelib/thread/qbasicatomic.h b/src/corelib/thread/qbasicatomic.h index e3fbe55890..6d061ea49a 100644 --- a/src/corelib/thread/qbasicatomic.h +++ b/src/corelib/thread/qbasicatomic.h @@ -19,8 +19,6 @@ QT_END_NAMESPACE #pragma qt_sync_stop_processing #endif -// New atomics - template class QBasicAtomicInteger {