struct fr_connection 'number' label

[Alan DeKok]

Aaron Hurt wrote:
> Should fr_connection->number match fr_connection_pool_t->count  … it
> seems as though number is being assigned the incremented value of count
> and number is a signed int whereas count is an unsigned int?  What
> happens if count grows larger than the max signed int?  Should both of
> these be changed to a long or unsigned long to reduce the chance of
> overflow to a near impossibility?

  If it opens 100 new connections a second, it will take a month to roll
over 2^32.

  Annoying, but not critical.  Making it a uint64_t would fix that for
all reasonable definitions of "forever".

  Alan DeKok.