summaryrefslogtreecommitdiffstats
path: root/examples/widgets/charactermap/mainwindow.cpp
diff options
context:
space:
mode:
authorThiago Macieira <thiago.macieira@intel.com>2021-03-21 13:57:48 -0700
committerQt Cherry-pick Bot <cherrypick_bot@qt-project.org>2021-04-20 19:33:00 +0000
commit8bae441d902f28db2f34a8eafff1d481ebc7d6e6 (patch)
tree079a0f7b871ea38e79a535ac865698933cf8bf13 /examples/widgets/charactermap/mainwindow.cpp
parentb1f956106cf0fc41eb12d76bb3625fae7687e5f3 (diff)
QUtf8StringView: ensure binary compatibility throughout 6.x
We can't depend on whether the user compiles with -std=c++17 or -std=c++20. So select what we can depend on and make that permanent. Prior to this change: $ cat /tmp/test.cpp #include <QtCore/QUtf8StringView> void f(QUtf8StringView) {} $ qcd include $ g++ -S -o - -I. /tmp/test.cpp | grep globl | c++filt .globl f(QBasicUtf8StringView<false>) $ g++ -fPIC -std=c++20 -S -o - -I. /tmp/test.cpp | grep globl | c++filt .globl f(QBasicUtf8StringView<true>) After this change, they're both "false". QUtf8StringView should have been a concrete class that derived from QBsicUtf8StringView<whichever> and inherited all its constructors. We'd cause ODR violations in C++20, but nothing worse than what we usually do for BC reasons. That solution is too late for Qt 6.x. Let's revisit in 7.0. Change-Id: I6bcbe88c072a438b8b4efffd166e77199ecb39e3 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io> (cherry picked from commit c7deb3400faa1d9389137fe63ffbffade8203850) Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
Diffstat (limited to 'examples/widgets/charactermap/mainwindow.cpp')
0 files changed, 0 insertions, 0 deletions