douglascraigschmidt/POSA-14 — pull request #21 by Sardtok (2014-05-25 17:14:03+00:00)
Title: Unneeded local variable added in SimpleAtomicLong

In week 3's skeleton for SimpleAtomicLong, you've added an unneeded local variable in the constructor. This can be confusing for new programmers who might think this is actually a needed variable.

Since the object reference has not yet been returned, and you shouldn't be leaking *this* in the constructor. The member variable, mValue, should not be accessible to any other threads yet, so race conditions should not be possible.

Also commented on in the course discussion board:
https://class.coursera.org/posa-002/forum/thread?thread_id=710