{"comment":{"commit_id":"01492deb9cdaae5acf0c2d009c13f3fc7d423d1c","position":49,"created_at":"2011-12-11T06:30:29Z","updated_at":"2011-12-11T06:30:29Z","user":{"gravatar_id":"2856d2acb4f3f2b6197a852250e9061b","id":1118286,"url":"https://api.github.com/users/nettoyeurny","avatar_url":"https://secure.gravatar.com/avatar/2856d2acb4f3f2b6197a852250e9061b?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"nettoyeurny"},"body":"You know, this syntactic sugar gives me the creeps.  Whenever I see it, I have the urge to go looking for overridden accessors, lest there be some unexpected side effect.  I know the arguments for using accessors, in particular the possibility of logging every access for debugging purposes, but I don't think that would help here because the new revision only uses these properties to hold values for the convenience of client code.  Internally, the class just sets the values upon configuration and then never looks at them again.","id":784695,"path":"objc/PdAudioController.m","line":93,"html_url":"https://github.com/libpd/libpd/commit/01492deb9c#commitcomment-784695","url":"https://api.github.com/repos/libpd/libpd/comments/784695"}}