bruns added inline comments.


> apol wrote in documenturldb.cpp:44
> Would it make sense to include a warning then? If it's a wrong input, we'll 
> need ways to debug it now that it won't be giving a backtrace.

This may be accessed quite frequently, so we would get a lot of warnings, 
spamming the journal, and slowing down the system.

A backtrace is useless here, as the code processes whatever it finds in the 
database. The interesting part is how the bad data ended up in the db.

Doing the checks here avoids spreading any existing corruption.

The long term solution is to:

1. Find the bugs leading to the DB corruption
2. Run lowlevel and highlevel sanity checks on existing databases

Both are on my TODO list, but this will take some time.

  R293 Baloo


To: bruns, #baloo, michaelh, #frameworks
Cc: apol, sitter, kde-frameworks-devel, broulik, #frameworks, ashaposhnikov, 
michaelh, astippich, spoorun, ngraham, bruns, abrahams

Reply via email to