Small changes accumulated while reading through a bunch of files in fdbrpc (mostly).
Throughout: add FIXME comments where it would be useful to have a comment explaining something or suggesting a change of some kind.
Several places: rename a variable to get a better name
A few places: delete code that has been commented out for > 5 years
A few places: delete useless or apparently out of date, incorrect comments
One place: delete a function that has no callers
One place: reorder methods, listing them in the order which users are to call them (i.e. the natural order for explanation)
One or more places: delete unneeded FIXME comments
network.h: put the function comments before the functions they describe
* put function comments before the function they describe
* add fixme comment
* setLocalAddress had an obviously broken function comment. Then I noticed nobody was calling it. So delete it
* QueueModel.h: list methods and comments in order that methods are to be called by users; remove a many years old commented out block of code
* fdbrpc.h: FIXMEs to request explanations of Public and Private
* Add a variety of FIXME comments to FlowTransport.cpp to be revisited later
* add FIXMEs for comment additions; remove useless or outdated comments; rename a variable for clarity; other very cosmetic updates
* add FIXMEs for suggested comments; rename a variable; delete one or more useless comments
* simplify some FIXME comments
* add one space to address CODE FORMAT CLEAN CI check
* Updates based on code review comments
* fix comment
* - Compare storage replicas on reads (in "loadBalance()")
* - Do consistency check on reads in loadbalance
* - Do replica consistency check in the case where loadBalance issues
requests to multiple storage servers
* - Address a state variable related bug
* - Code formatting
* - API simplification
* - Simplify code
* - Code formatting
* - Address a review comment
gxglass