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

Eradicate QT_STRINGVIEW_LEVEL

Browse Source 
It was designed to gauge the impact of replacing QString functions
with QStringView ones, as opposed to just overloading, but that never
came to pass, for two reasons:

1. We never got to implement a full set of overloads with the pattern
   in a large enough library (QtCore) for results to matter.

2. QStringView doesn't accept everything that QString accepts, making
   it unsuitable as a drop-in replacement (QAnyStringView has that
   property, though, but we never added a QT_ANYSTRINGVIEW_LEVEL
   macro).

So, R.I.P. QT_STRINGVIEW_LEVEL.

Fixes: QTBUG-100861
Change-Id: I0a9318177acb295e8eeeddfb345449112070c597
Reviewed-by: Sona Kurazyan <sona.kurazyan@qt.io>
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
(cherry picked from commit 046a5a8f4f49944e6bf94cded52f88535d16bb3f)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
This commit is contained in:
Marc Mutz 2022-07-18 10:24:37 +02:00 committed by Qt Cherry-pick Bot
parent 0102ab6cd3
commit a905c3e683
1 changed files with 0 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/corelib/text/qstringview.h
View File

@ -4,13 +4,6 @@
#ifndef QSTRINGVIEW_H
#define QSTRINGVIEW_H
/*
Obsolete.
*/
#ifndef QT_STRINGVIEW_LEVEL
# define QT_STRINGVIEW_LEVEL 1
#endif
#include <QtCore/qchar.h>
#include <QtCore/qbytearray.h>
#include <QtCore/qstringliteral.h>