Skip to content

Fix QString::SplitBehavior deprecation, replace with Qt::#1302

Merged
digisomni merged 1 commit intovircadia:masterfrom
daleglass:qt5-fix-qstring-splitbehavior
Aug 12, 2021
Merged

Fix QString::SplitBehavior deprecation, replace with Qt::#1302
digisomni merged 1 commit intovircadia:masterfrom
daleglass:qt5-fix-qstring-splitbehavior

Conversation

@daleglass
Copy link
Contributor

Fixes a bunch of this:

‘QStringList QString::split(const QRegExp&, QString::SplitBehavior) const’ is deprecated: Use Qt::SplitBehavior variant instead [-Wdeprecated-declarations]

@digisomni digisomni added this to the 2021.2.0 Selene Release milestone Aug 7, 2021
@ctrlaltdavid ctrlaltdavid added CR Approved At least one code reviewer has approved the PR. and removed needs CR (code review) labels Aug 7, 2021
@digisomni digisomni removed the needs testing (QA) The PR is ready for testing label Aug 7, 2021
@digisomni digisomni merged commit 6eb7b9e into vircadia:master Aug 12, 2021
@ArcadeFever
Copy link
Contributor

ArcadeFever commented Dec 6, 2021

Looks like this PR didn't build on Android...likely because wrong QT version? How long has Android build been broken?

@JulianGro
Copy link
Contributor

Android builds have been broken for quite some time now. I would guess around two months.
Dale tried fixing that but got a bit stuck: #1280

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix CR Approved At least one code reviewer has approved the PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants