lbeltrame accepted this revision.
lbeltrame added a comment.
This revision is now accepted and ready to land.


  Looks OK to me with some changes (for readability).

INLINE COMMENTS

> fileindexerconfig.h:50
>      /**
> -     * The folders to search for files to analyze. Cached and cleaned up.
> +     * Folders to search for files to analyze and index.
> +     * \return list of paths.

I'd put "index and analyze". Seems nitpicky, but reflects what Baloo does 
(index first, extraction later).

> fileindexerconfig.h:57
> +     * Folders that are excluded from indexing.
> +     * (It is possible to include descendants of an excluded folder again
> +     * and they will be indexed.)

"Descendant folders of an excluded folder can be added and they will be 
indexed."

REPOSITORY
  R293 Baloo

BRANCH
  publicdescriptions (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D10662

To: michaelh, #baloo, lbeltrame
Cc: lbeltrame, adridg, #frameworks, ashaposhnikov, michaelh, spoorun, 
nicolasfella, alexeymin

Reply via email to