{"comment":{"url":"https://api.github.com/repos/danieldresser/cortex/comments/12128131","html_url":"https://github.com/danieldresser/cortex/commit/0bc64aed4fbac9f73f8c137bdae5e490fe3ecf3f#commitcomment-12128131","id":12128131,"user":{"login":"johnhaddon","id":1133871,"avatar_url":"https://avatars.githubusercontent.com/u/1133871?v=3","gravatar_id":"","url":"https://api.github.com/users/johnhaddon","html_url":"https://github.com/johnhaddon","followers_url":"https://api.github.com/users/johnhaddon/followers","following_url":"https://api.github.com/users/johnhaddon/following{/other_user}","gists_url":"https://api.github.com/users/johnhaddon/gists{/gist_id}","starred_url":"https://api.github.com/users/johnhaddon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/johnhaddon/subscriptions","organizations_url":"https://api.github.com/users/johnhaddon/orgs","repos_url":"https://api.github.com/users/johnhaddon/repos","events_url":"https://api.github.com/users/johnhaddon/events{/privacy}","received_events_url":"https://api.github.com/users/johnhaddon/received_events","type":"User","site_admin":false},"position":48,"line":48,"path":"include/IECore/DataAlgo.h","commit_id":"0bc64aed4fbac9f73f8c137bdae5e490fe3ecf3f","created_at":"2015-07-13T08:27:15Z","updated_at":"2015-07-13T08:27:15Z","body":"Should take `Data *data` - we don't need to hold a reference to the data after the function returns. Perhaps the argument types are motivated by issues with the interface for `despatchTypedData()`? If that's the case I'd rather use casts behind the scenes so that the API is right here, and then fix up `despatchTypedData()` in the future."}}