Revisit ServerIdentifier::hash()

 
Currently it is implemented as

   unsigned hash() const { return netAddress; }

which may be a problem now that the ip mapping hash table directly uses ServerIdentitiers.  Only the bottom byte is used by lookup():
**
 

unsigned hash = ip.fasthash() & 0xff;  

 which is likely be have reasonably well on most systems, so not a high priority.