Правильно ли написан код, выдающий уникальные цифры, основанные на таймере?

Question

[Mercury13]

// timer — общий на всю прогу atomic<long long>
// Можно рассчитывать только на 64-битную архитектуру

long long newTime = someUtcTime();
long long oldTime = timer;
do {
  newTime = std::max(newTime, oldTime + 1);     // должен быть больше того, что имеется
} while (!timer.compare_exchange_strong(&oldTime, newTime))
return (oldTime = newTime);

jcmvbkbc, спасибо за коррективы

Comments

[res2001]

Похоже, если функция действительно вызывается в разных потоках, то она может долго ждать в этом цикле пока в timer снова появится значение oldTime.
Вообще, не понятно, если вы сохраняете предыдущее значение в локальную память потока, то зачем вам этот atomic нужен?

[jcmvbkbc]

она может долго ждать в этом цикле пока в timer снова появится значение oldTime.

res2001, если oldTime != time, значение oldTime будет обновлено текущим значением time.

[res2001]

jcmvbkbc, Точно. Что-то давно не использовал CASы :)
Тогда вроде все должно работать.