summaryrefslogtreecommitdiffstats
path: root/src/widgets/kernel/qwidget.cpp
diff options
context:
space:
mode:
authorVolker Hilsheimer <volker.hilsheimer@qt.io>2023-04-07 11:58:38 +0200
committerVolker Hilsheimer <volker.hilsheimer@qt.io>2023-06-21 00:30:41 +0200
commit3f72b0d5fc70d3cf7daa4badccd5a40fc8b0726a (patch)
tree4d2a0d0c6874c4014f04940ab7470c402236d3f9 /src/widgets/kernel/qwidget.cpp
parent054a64ce67421b213df73a9bcfda3fa6e9c6e64f (diff)
QSqlResult: remove bad API returning non-const reference
QSqlResult::boundValues is a const member function, but returned a non- const reference to a QList<QVariant>. This is a bad and potentially dangerous API, as callers can modify the list stored in QSqlResult. Move that API into the removed_api translation unit, remove it from Qt 6.6 on and replace it with two suitable overloads where the const version returns a QVariantList by value, and the non-const overload returns a mutable reference. Driver implementations that used to call the const overload to get a mutable reference are now calling the non-const overload instead (those calls are all made in the non-const exec() or equivalent driver implementations). As a drive-by, replace "vector" with "list" in the documentation. Pick-to: 6.6 Change-Id: I6e4fd8f5749b939cdb609bf5876735e9b30b2b5a Reviewed-by: Marc Mutz <marc.mutz@qt.io>
Diffstat (limited to 'src/widgets/kernel/qwidget.cpp')
0 files changed, 0 insertions, 0 deletions