Review Request: Ask the user whether to clean DUChain repository after multiple crashes.

Dmitry Risenberg dmitry.risenberg at gmail.com
Tue Mar 8 14:25:47 UTC 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100822/
-----------------------------------------------------------

Review request for KDevelop.


Summary
-------

Got bitten by the code in ItemRepositoryRegistry::open a few times while debugging, when my repository was cleaned and total reparsing was made. The patch adds a message box which allows the user to choose between cleaning the repository and leaving it as is.

The bigger question - is this code still useful? I haven't experienced any DUChain-related crashes, while crashes for other reasons still happen. So the user might get a full reparsing if some other part of KDevelop misbehaves.


Diffs
-----

  language/duchain/repositories/itemrepository.cpp e603b222330faf994525837e8d0d3c3931dd61c4 

Diff: http://git.reviewboard.kde.org/r/100822/diff


Testing
-------


Thanks,

Dmitry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20110308/a99c35ce/attachment.html>


More information about the KDevelop-devel mailing list