poboiko added a comment.
Maybe we should print also a suggestion to user, something like (maybe
rephrase it better)
WARNING:
Looks like your index is corrupted.
We suggest you to run `balooctl disable && balooctl disable` to wipe it and
rebuild from scratch
so they won't have to google what to do (there's still not much can be done
in that case)
BTW, does it also wipe tags and other user-provided metadata?
INLINE COMMENTS
> bruns wrote in main.cpp:204
> See `word[0]` access directly after.
> I have fixed to many "shouldn't be" errors/crashes due to corrupt DB values
> in the past.
OK, you're right, I guess.
> bruns wrote in main.cpp:211
> This is the exact case I have had - "X<garbage">.
> The code after (`word.indexOf('-', 2)`) requires a check for length >= 3 here
> (code correctness), semantics require >= 4.
But still, if the term is short (namely, length < 4), we will either won't have
"-" (this corresponds to `posOfNonNumeric < 0`, and that's `X<garbage>`), or it
will be the last symbol (something like `X1-` - which is `posOfNonNumeric+1 ==
word.length()`).
> bruns wrote in main.cpp:218
> I am not really sure:
> balooctl -x is a quite low level debug tool. This is a diagnostic message
> only printed in case of DB errors. Translated strings make search on the web
> harder.
On the one hand, you're right. On the other hand, the output should be
consistent. Other messages here are translated.
REPOSITORY
R293 Baloo
REVISION DETAIL
https://phabricator.kde.org/D15826
To: bruns, #baloo, #frameworks, poboiko
Cc: anthonyfieroni, kde-frameworks-devel, ashaposhnikov, michaelh, astippich,
spoorun, ngraham, bruns, abrahams
