zeasy/EasyBinding — pull request #1 by th-in-gs (2012-05-14 23:22:53+00:00)
Title: Rename newValue to valueNew and oldValue to valueOld.

By Cocoa naming conventions, a method named 'newValue' should return a +1 retain count object, because it starts with the word 'new', signifying object creation. This is not the intention here though - the 'newValue' method is an accessor for the new value, not a creation method.

This is a problem when interoperating between ARC and non-ARC code, because ARC expects a +1 retain-count obejct to be returned by the 'newValue' method, but non-ARC code, written by intelligent people who can reason about the method's purpose, probably does not.

To fix the issue, I renamed the 'newValue' property to 'valueNew'. Because the 'new' is now at the _end_ of the accessor name, it no longer conforms to the naming convention for object creation, and ARC does not get confused (and neither do humans, hopefully).

'oldValue' does not, of course, exhibit the same problem, but I also renamed it, for symmetry.