10 Dec
2013
10 Dec
'13
5:29 p.m.
Hi, Sergey!
Not a review, only a comment on TDC_atomic part:
1. This is perfectly generally useful, not TDC specific at all.
I'd consider moving it to my_atomic.h under #ifdef __cplusplus
2. These are supposed to be trivial inlined functions. I don't think you
should use inheritance and virtual methods here. Everything should be
in the template.
I understand that function names (add64/add32/etc) are a bit tricky.
Two approaches: pass them as template parameters (like in
template