diff options
author | Marc Mutz <[email protected]> | 2025-06-02 17:25:40 +0200 |
---|---|---|
committer | Marc Mutz <[email protected]> | 2025-06-06 22:16:36 +0200 |
commit | a97ac8507e39a421f530f5ad612a7e4bb58f6718 (patch) | |
tree | 855595d187f000c2d7b46528121b8d8867eecffb /src/sql/models/qsqlquerymodel.cpp | |
parent | 99ad6a51a595430c75323c4d5c2ecbc7fbd8d7f0 (diff) |
The kludge that was added to argumentTypesFromString() for Qt 6 to
support merging QVector into QList—specifically, the replacement of
QVector< with QList<—was unfortunately not removed before the Qt 6.0
release. As a kludge, it has unintended consequences, as Thiago
pointed out in a comment on a related patch.
This change adds a reproducer that demonstrates cases where the
kludge causes correct code to fail incorrectly. We need this test to
ensure that we do not silently change behavior when deprecating and
eventually removing the kludge.
Adapt the MyQList normalization test, which was carefully written to
avoid hitting the kludge, to use template arguments. This will allow
it to trigger the buggy code path and avoid confusing the reader with
the mention of a non-template MyQList, given that MyQList is now a
template.
Reported-by: Thiago Macieira <[email protected]>
Pick-to: 6.10 6.9 6.8 6.5
Task-number: QTBUG-135572
Change-Id: I91d769d494489fe63dbbb67f849b78fc7aa39ec6
Reviewed-by: Ahmad Samir <[email protected]>
Diffstat (limited to 'src/sql/models/qsqlquerymodel.cpp')
0 files changed, 0 insertions, 0 deletions