{"action":"created","issue":{"url":"https://api.github.com/repos/mpicbg-scicomp/gearshifft/issues/52","repository_url":"https://api.github.com/repos/mpicbg-scicomp/gearshifft","labels_url":"https://api.github.com/repos/mpicbg-scicomp/gearshifft/issues/52/labels{/name}","comments_url":"https://api.github.com/repos/mpicbg-scicomp/gearshifft/issues/52/comments","events_url":"https://api.github.com/repos/mpicbg-scicomp/gearshifft/issues/52/events","html_url":"https://github.com/mpicbg-scicomp/gearshifft/issues/52","id":190043858,"number":52,"title":"Unify interface naming style?","user":{"login":"tdd11235813","id":16594217,"avatar_url":"https://avatars.githubusercontent.com/u/16594217?v=3","gravatar_id":"","url":"https://api.github.com/users/tdd11235813","html_url":"https://github.com/tdd11235813","followers_url":"https://api.github.com/users/tdd11235813/followers","following_url":"https://api.github.com/users/tdd11235813/following{/other_user}","gists_url":"https://api.github.com/users/tdd11235813/gists{/gist_id}","starred_url":"https://api.github.com/users/tdd11235813/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tdd11235813/subscriptions","organizations_url":"https://api.github.com/users/tdd11235813/orgs","repos_url":"https://api.github.com/users/tdd11235813/repos","events_url":"https://api.github.com/users/tdd11235813/events{/privacy}","received_events_url":"https://api.github.com/users/tdd11235813/received_events","type":"User","site_admin":false},"labels":[{"id":388966765,"url":"https://api.github.com/repos/mpicbg-scicomp/gearshifft/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":{"login":"tdd11235813","id":16594217,"avatar_url":"https://avatars.githubusercontent.com/u/16594217?v=3","gravatar_id":"","url":"https://api.github.com/users/tdd11235813","html_url":"https://github.com/tdd11235813","followers_url":"https://api.github.com/users/tdd11235813/followers","following_url":"https://api.github.com/users/tdd11235813/following{/other_user}","gists_url":"https://api.github.com/users/tdd11235813/gists{/gist_id}","starred_url":"https://api.github.com/users/tdd11235813/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tdd11235813/subscriptions","organizations_url":"https://api.github.com/users/tdd11235813/orgs","repos_url":"https://api.github.com/users/tdd11235813/repos","events_url":"https://api.github.com/users/tdd11235813/events{/privacy}","received_events_url":"https://api.github.com/users/tdd11235813/received_events","type":"User","site_admin":false},"assignees":[{"login":"tdd11235813","id":16594217,"avatar_url":"https://avatars.githubusercontent.com/u/16594217?v=3","gravatar_id":"","url":"https://api.github.com/users/tdd11235813","html_url":"https://github.com/tdd11235813","followers_url":"https://api.github.com/users/tdd11235813/followers","following_url":"https://api.github.com/users/tdd11235813/following{/other_user}","gists_url":"https://api.github.com/users/tdd11235813/gists{/gist_id}","starred_url":"https://api.github.com/users/tdd11235813/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tdd11235813/subscriptions","organizations_url":"https://api.github.com/users/tdd11235813/orgs","repos_url":"https://api.github.com/users/tdd11235813/repos","events_url":"https://api.github.com/users/tdd11235813/events{/privacy}","received_events_url":"https://api.github.com/users/tdd11235813/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2016-11-17T13:15:25Z","updated_at":"2016-11-17T14:21:01Z","closed_at":null,"body":"C or C++ style?\r\n```C++\r\n  void malloc(); // or allocate?\r\n  size_t getAllocSize(); // or get_alloc_size?\r\n  size_t getTransferSize();\r\n  size_t getPlanSize();\r\n  void init_forward(); // or initForward?\r\n  void init_backward();\r\n  void execute_forward(); // or executeForward?\r\n  void execute_backward();\r\n  template<typename THostData>                                                                                                                                             \r\n  void upload(THostData* input);\r\n  template<typename THostData>                                                                                                                                             \r\n  void download(THostData* output);\r\n  void destroy();\r\n```\r\nI would suggest to stick with doSomething instead of do_something ...\r\n\r\nEdit:\r\n`TPlan` might be the wrong name for user FFT implementation, have to find a better name."},"comment":{"url":"https://api.github.com/repos/mpicbg-scicomp/gearshifft/issues/comments/261258202","html_url":"https://github.com/mpicbg-scicomp/gearshifft/issues/52#issuecomment-261258202","issue_url":"https://api.github.com/repos/mpicbg-scicomp/gearshifft/issues/52","id":261258202,"user":{"login":"tdd11235813","id":16594217,"avatar_url":"https://avatars.githubusercontent.com/u/16594217?v=3","gravatar_id":"","url":"https://api.github.com/users/tdd11235813","html_url":"https://github.com/tdd11235813","followers_url":"https://api.github.com/users/tdd11235813/followers","following_url":"https://api.github.com/users/tdd11235813/following{/other_user}","gists_url":"https://api.github.com/users/tdd11235813/gists{/gist_id}","starred_url":"https://api.github.com/users/tdd11235813/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tdd11235813/subscriptions","organizations_url":"https://api.github.com/users/tdd11235813/orgs","repos_url":"https://api.github.com/users/tdd11235813/repos","events_url":"https://api.github.com/users/tdd11235813/events{/privacy}","received_events_url":"https://api.github.com/users/tdd11235813/received_events","type":"User","site_admin":false},"created_at":"2016-11-17T14:21:01Z","updated_at":"2016-11-17T14:21:01Z","body":"ah I see, probably you are right. I just thought the style was to \"C-ish\" or somehow outdated^^. \r\nBut I agree, lowercase is more readable, so let us choose this.\r\nuhm, I am not aware of a clang style, but for local function arguments I began to use `_variable` and for members `member_` style, what do you suggest?"}}
{"actor":{"display_login":"tdd11235813"}}