[Swan-dev] *cmp is a little harder to read
D. Hugh Redelmeier
hugh at mimosa.com
Thu Nov 2 01:04:40 UTC 2017
I'm concerned about eb8a4265ffd98aa2f41a5b98c83876fcbfd19671
timers: add more generic deltatime_cmp()
expect use to follow the convention: deltatime_cmp(a, b) OP 0
for instance deltatime_cmp(a, b) <= 0; like strcmp() et.al.
It is true that *cmp is a more general primative. But it is also
harder to read code using it.
So: it is fine to add deltatime_cmp but I disagree with this comment:
+bool deltaless(deltatime_t a, deltatime_t b); /* obsolete; use deltatime_cmp() */
Uses of deltaless seem clearer to me than their replacement.
More information about the Swan-dev
mailing list