1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

QPostEventList: use qsizetype to count posted events

Browse Source 
It's highly unlikely you're going to have more than 2 billion pending
events for a thread, but this is the right thing to do.

Change-Id: Ia162b024815c3b970d71fffd3bc022621b2e07b4
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Reviewed-by: Ahmad Samir <a.samirh78@gmail.com>
(cherry picked from commit 67a71d71093b50d0db172b92aa1b548a1a2bd959)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
This commit is contained in:
Thiago Macieira 2024-11-13 11:44:57 -08:00 committed by Qt Cherry-pick Bot
parent f0c7c66b21
commit 027ee82051
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/corelib/kernel/qobject.cpp
View File

@ -1738,8 +1738,8 @@ void QObjectPrivate::setThreadData_helper(QThreadData *currentData, QThreadData
}
// move posted events
int eventsMoved = 0;
for (int i = 0; i < currentData->postEventList.size(); ++i) {
qsizetype eventsMoved = 0;
for (qsizetype i = 0; i < currentData->postEventList.size(); ++i) {
const QPostEvent &pe = currentData->postEventList.at(i);
if (!pe.event)
continue;

2
src/corelib/thread/qthread.cpp
View File

@ -82,7 +82,7 @@ QThreadData::~QThreadData()
thread.storeRelease(nullptr);
delete t;
for (int i = 0; i < postEventList.size(); ++i) {
for (qsizetype i = 0; i < postEventList.size(); ++i) {
const QPostEvent &pe = postEventList.at(i);
if (pe.event) {
--pe.receiver->d_func()->postedEvents;