Improved LRUCache documentation.

Documented that GetterFunctions and RemovalCallbacks must never access the LRUCache itself. This is because that could result in deadlocks as locks are held while they are called.

Fixed formatting to improve clarity.
Removed bogus assertion from LRUCache.

As is clear from looking at the if() above, it's perfectly possible to arrive at that point with status==TooCostly.