bruns created this revision. Restricted Application added projects: Frameworks, Baloo. Restricted Application added a subscriber: Frameworks. bruns requested review of this revision.
REVISION SUMMARY The Term(Term, Operator, Term) constructor is identical to the three separate calls, but saves two separate function calls. Signed-off-by: Stefan Brüns <stefan.bru...@rwth-aachen.de> Try to merge terms with identical operations (a AND b) AND (c OR d) can be merged as (a AND b AND (c OR D), i.e. 2 instead of 3 intermediate AND/OR terms. This is especially useful if terms are combined with a && b && c && d, which can be combined into one AND term with 4 subterms, instead of 3 AND terms with 2 subterms each (((a AND b) AND c) AND d). Signed-off-by: Stefan Brüns <stefan.bru...@rwth-aachen.de> REPOSITORY R293 Baloo BRANCH baloo_term_cleanup REVISION DETAIL https://phabricator.kde.org/D11826 AFFECTED FILES src/lib/term.cpp src/lib/term.h To: bruns Cc: #frameworks, ashaposhnikov, michaelh, astippich, spoorun, ngraham, alexeymin
