{"comment":{"url":"https://api.github.com/repos/opensim-org/opensim-core/comments/12486428","html_url":"https://github.com/opensim-org/opensim-core/commit/3441598f1659a2e4ff2018fc05f97f1340bd35e8#commitcomment-12486428","id":12486428,"user":{"login":"sherm1","id":4088016,"avatar_url":"https://avatars.githubusercontent.com/u/4088016?v=3","gravatar_id":"","url":"https://api.github.com/users/sherm1","html_url":"https://github.com/sherm1","followers_url":"https://api.github.com/users/sherm1/followers","following_url":"https://api.github.com/users/sherm1/following{/other_user}","gists_url":"https://api.github.com/users/sherm1/gists{/gist_id}","starred_url":"https://api.github.com/users/sherm1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sherm1/subscriptions","organizations_url":"https://api.github.com/users/sherm1/orgs","repos_url":"https://api.github.com/users/sherm1/repos","events_url":"https://api.github.com/users/sherm1/events{/privacy}","received_events_url":"https://api.github.com/users/sherm1/received_events","type":"User","site_admin":false},"position":9,"line":60,"path":"OpenSim/Examples/Plugins/CoupledBushingForceExample/CoupledBushingForce.cpp","commit_id":"3441598f1659a2e4ff2018fc05f97f1340bd35e8","created_at":"2015-08-03T04:28:19Z","updated_at":"2015-08-03T04:28:19Z","body":"Looks good! Minor quibble: the parameter name `frame1_name` doesn't match OpenSim's naming conventions. Should be `frame1Name` or something like that. I think we reserved underscores for use in property names and automatically-generated methods involving properties. Same for `frame1_offset`."}}