Refactor ShutdownSocketSet atomic state machine
Summary:
[Folly] Refactor `ShutdownSocketSet` atomic state machine.
* Format.
* Use `while` over `goto`.
* Avoid `memory_order_acq_rel` as the single-argument memory order; some platforms fail to build that.
* Use `memory_order_relaxed` for all atomic operations because stronger memory orders are not actually required: the atomic state never serves as a barrier for synchronizing loads and stores of associated data because there is no associated data.
Reviewed By: davidtgoldblatt
Differential Revision:
D6058292
fbshipit-source-id:
d45d7fcfa472e6e393a5f980e75ad9ea3358bab3