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

Restore pre-Qt6 QList::fill() behavior

Browse Source 
Somehow QList::fill(t, newSize) introduced a regression in Qt6:
when newSize < QList::size() we should resize to the newSize.
This is aligned with QVector::fill() in 5.15 and std::vector::assign()

While 6.0 is already out, picking it to 6.0.x could save someone who
haven't migrated yet as well as fix some accidental bugs in Qt's code

[ChangeLog][QtCore][QList] Fixed QList::fill() regression introduced in
6.0: calling fill() with size < current list size wouldn't truncate the
list

Fixes: QTBUG-91042
Change-Id: Ic166e2c5e42390b61df1030f7c705e344433f7f2
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
(cherry picked from commit 6512a7fc642c65455db770385c67cfa6d71c294c)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
This commit is contained in:
Andrei Golubev 2021-02-09 14:03:04 +01:00 committed by Qt Cherry-pick Bot
parent 98e9275112
commit 249b310046
2 changed files with 67 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/corelib/tools/qlist.h
View File

@ -845,8 +845,11 @@ inline QList<T> &QList<T>::fill(parameter_type t, qsizetype newSize)
// we're detached
const T copy(t);
d->assign(d.begin(), d.begin() + qMin(size(), newSize), t);
if (newSize > size())
if (newSize > size()) {
d->copyAppend(newSize - size(), copy);
} else if (newSize < size()) {
d->truncate(newSize);
}
}
return *this;
}

70
tests/auto/corelib/tools/qlist/tst_qlist.cpp
View File

@ -255,7 +255,9 @@ private slots:
void fillInt() const;
void fillMovable() const;
void fillCustom() const;
void fillDetaches() const;
void fillDetachInt() const;
void fillDetachMovable() const;
void fillDetachCustom() const;
void first() const;
void fromListInt() const;
void fromListMovable() const;
@ -362,6 +364,7 @@ private:
template<typename T> void erase(bool shared) const;
template<typename T> void eraseReserved() const;
template<typename T> void fill() const;
template<typename T> void fillDetach() const;
template<typename T> void fromList() const;
template<typename T> void insert() const;
template<typename T> void qhash() const;
@ -1442,6 +1445,10 @@ void tst_QList::fill() const
<< SimpleValue<T>::at(2) << SimpleValue<T>::at(2)
<< SimpleValue<T>::at(2) << SimpleValue<T>::at(2)
<< SimpleValue<T>::at(2) << SimpleValue<T>::at(2));
// make sure it can resize to smaller size as well
myvec.fill(SimpleValue<T>::at(3), 2);
QCOMPARE(myvec, QList<T>() << SimpleValue<T>::at(3) << SimpleValue<T>::at(3));
}
void tst_QList::fillInt() const
@ -1463,14 +1470,63 @@ void tst_QList::fillCustom() const
QCOMPARE(instancesCount, Custom::counter.loadAcquire());
}
void tst_QList::fillDetaches() const
template<typename T>
void tst_QList::fillDetach() const
{
QList<int> test = { 1, 2, 3 };
QList<int> copy = test;
copy.fill(42);
// detaches to the same size
{
QList<T> original = { SimpleValue<T>::at(1), SimpleValue<T>::at(1), SimpleValue<T>::at(1) };
QList<T> copy = original;
copy.fill(SimpleValue<T>::at(2));
QCOMPARE(test, QList<int>({1, 2, 3}));
QCOMPARE(copy, QList<int>({42, 42, 42}));
QCOMPARE(original,
QList<T>({ SimpleValue<T>::at(1), SimpleValue<T>::at(1), SimpleValue<T>::at(1) }));
QCOMPARE(copy,
QList<T>({ SimpleValue<T>::at(2), SimpleValue<T>::at(2), SimpleValue<T>::at(2) }));
}
// detaches and grows in size
{
QList<T> original = { SimpleValue<T>::at(1), SimpleValue<T>::at(1), SimpleValue<T>::at(1) };
QList<T> copy = original;
copy.fill(SimpleValue<T>::at(2), 5);
QCOMPARE(original,
QList<T>({ SimpleValue<T>::at(1), SimpleValue<T>::at(1), SimpleValue<T>::at(1) }));
QCOMPARE(copy,
QList<T>({ SimpleValue<T>::at(2), SimpleValue<T>::at(2), SimpleValue<T>::at(2),
SimpleValue<T>::at(2), SimpleValue<T>::at(2) }));
}
// detaches and shrinks in size
{
QList<T> original = { SimpleValue<T>::at(1), SimpleValue<T>::at(1), SimpleValue<T>::at(1) };
QList<T> copy = original;
copy.fill(SimpleValue<T>::at(2), 1);
QCOMPARE(original,
QList<T>({ SimpleValue<T>::at(1), SimpleValue<T>::at(1), SimpleValue<T>::at(1) }));
QCOMPARE(copy, QList<T>({ SimpleValue<T>::at(2) }));
}
}
void tst_QList::fillDetachInt() const
{
fillDetach<int>();
}
void tst_QList::fillDetachMovable() const
{
const int instancesCount = Movable::counter.loadAcquire();
fillDetach<Movable>();
QCOMPARE(instancesCount, Movable::counter.loadAcquire());
}
void tst_QList::fillDetachCustom() const
{
const int instancesCount = Custom::counter.loadAcquire();
fillDetach<Custom>();
QCOMPARE(instancesCount, Custom::counter.loadAcquire());
}
void tst_QList::first() const