D11826: Simplify Term operator&& and ||

[Stefan Brüns]

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.bruens at 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.bruens at 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180331/df4d6251/attachment-0001.html>