As requested in D28196 this patch handles the early return and the formatting.
Additionally QRegularExpression has been used instead of QRegExp and QStringLiteral instead of QString.
Details
- Reviewers
bruns broulik - Group Reviewers
Plasma - Commits
- R120:e518d45a4d61: BookmarksRunner: Early return and formatting
Should compile and work exactly as before.
To test migration to QRegularExpression:
create multiple profiles
delete the dbfile entry from the general group in .config/kdeglobals
restart krunner and make sure that the value of the dbfile entry is the path to the default profile.
The bookmarks should also be displayed.
Diff Detail
- Repository
- R120 Plasma Workspace
- Branch
- bookmarksrunner_firefox_fix_prep (branched from master)
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 24555 Build 24573: arc lint + arc unit
Should compile and work exactly as before.
Please always test QRegularExpression migration, I've seen just too many regressions when this wasn't done carefully
runners/bookmarks/browsers/firefox.cpp | ||
---|---|---|
171 | The default argument is sufficient to resolve the type, readEntry("foo", QString()) is enough |
Implement requested changes.
PS: I had already tested the migration locally and added it to the test plan.
runners/bookmarks/browsers/firefox.cpp | ||
---|---|---|
198 | Can you leave this in, and even promote it to an active qWarning* - if we have a db and profiles file, but can't determine the default profile, something is definitely wrong.
| |
214 | This can IMHO also be promoted to an active warning. |
runners/bookmarks/browsers/firefox.cpp | ||
---|---|---|
166 | Thanks, though I was wrong about "no categories yet". #include "bookmarks_debug" ... qCWarning(RUNNER_BOOKMARKS) << ... |
Should this(and the following patches) go to a stable branch, because they fix a bug that has been reported (418526)?