Skip to content
Commit 8c7a95cc authored by Harald Sitter's avatar Harald Sitter 🐧
Browse files

take matching logic out of match method and put it into a class

Summary:
refactor and qcdebug

::match was a super long spagetthi with multiple sub-matches.
This was supposedly primarily because it needed to be stateful across all sub-matches. To deal with this there's a new class which is taking care of the stateful service finding, this class is comprised of multiple matching functions that contribute to the full match set.
Good enough readability for now. Classes for this would be nicer though.

Also add qcdebug for this runner so we can get a better idea of why results are the way they are. Defaults to warning so it's silent by default.

Test Plan:
Careful refactoring so hopefully no problems.

- still matches binary names as best match
- still matches comments
- still matches generic names
- still matches names

Reviewers: broulik

Reviewed By: broulik

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D3687
parent bf1dfd4d
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment