kwinglplatformtest still passes.
Details
Details
- Reviewers
apol - Group Reviewers
KWin - Commits
- R108:9755ef20e585: [kwineffects] Port GLPlatform to QRegularExpression
Diff Detail
Diff Detail
- Repository
- R108 KWin
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
libkwineffects/kwinglplatform.cpp | ||
---|---|---|
97 | It could make sense to return a QStringRef, we'd get to save a bunch of string allocations and copies. |
libkwineffects/kwinglplatform.cpp | ||
---|---|---|
97 | Yes, it would be great, but could we maybe do it in another patch? QString and QStringRef have minor API differences so it's not just about replacing a few QStrings by QStringRefs. Such a change would look unrelated if you ask me. |
libkwineffects/kwinglplatform.cpp | ||
---|---|---|
97 | Yes, the patch is accepted, feel free to land. |