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