Add some logging
Add more information to logging (toString() provides more than the nickname) and make it sound less like we are finished yet
Fix assert() failure in beginTrustListImport() by moving the call to begin..() to the proper location
Comment out bogus assert()s and explain why they won't work
Add some assert()s
Finish repairing of restoreOwnIdentity:
- Instead of using deleteWithoutCommit(oldIdentity), inline the code for
deleting everything related to it.
- Use removeTrustWithoutCommit() instead of deleting all given trusts
manually to guarantee proper updating of the score tree. Notice that
the current implementation of removeTrustWithoutCommit is dumb, it will
schedule a full score recomputation instead of incrementally updating
the tree. I hope that the future incremental implementation does not
break restoreOwnIdentity. But even if it does, the
assert(computeAllScoresWithoutCommit()) in finishTrustListImport will
detect a broken tree.