{"issue":{"state":"open","pull_request":{"diff_url":null,"html_url":null,"patch_url":null},"user":{"avatar_url":"https://secure.gravatar.com/avatar/f118c0a737f68e86142684f57698a2bc?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"f118c0a737f68e86142684f57698a2bc","url":"https://api.github.com/users/malcolmhumphreys","login":"malcolmhumphreys","id":195463},"milestone":null,"updated_at":"2011-10-28T03:07:27Z","assignee":null,"labels":[{"name":"API Change","color":"ff7878","url":"https://api.github.com/repos/imageworks/OpenColorIO/labels/API+Change"}],"comments":1,"closed_at":null,"url":"https://api.github.com/repos/imageworks/OpenColorIO/issues/171","body":"Noticed a little API inconsistency between CDLTransform and MatrixTransform.\r\n\r\nbool CDLTransform::equals(const ConstCDLTransformRcPtr & other) const\r\nvs\r\nbool MatrixTransform::equals(const MatrixTransform & other) const\r\n\r\n\r\nI would think MatrixTransform would need to look like this:\r\n\r\nbool MatrixTransform::equals(const ConstMatrixTransformRcPtr & other) const","html_url":"https://github.com/imageworks/OpenColorIO/issues/171","title":"transform *::equals API inconsistency","number":171,"id":2067031,"created_at":"2011-10-27T11:57:00Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/d720dfcb6355d65e566af8fc68c36aca?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"d720dfcb6355d65e566af8fc68c36aca","url":"https://api.github.com/users/jeremyselan","login":"jeremyselan","id":240544},"updated_at":"2011-10-28T03:07:26Z","url":"https://api.github.com/repos/imageworks/OpenColorIO/issues/comments/2552076","body":"agreed.   tagging as 'API' (non-binary compatible), so it'll be awhile before we can safely remove it.  But I'm happy to add the additional signature in the meantime.  Do you have a preference?","id":2552076,"created_at":"2011-10-28T03:07:26Z"}}