{"comment":{"url":"https://api.github.com/repos/josevalim/otp/comments/9817402","html_url":"https://github.com/josevalim/otp/commit/309079d194ea4c012a22c56f06cc9571a52b99b3#commitcomment-9817402","id":9817402,"user":{"login":"bjorng","id":74840,"avatar_url":"https://avatars.githubusercontent.com/u/74840?v=3","gravatar_id":"","url":"https://api.github.com/users/bjorng","html_url":"https://github.com/bjorng","followers_url":"https://api.github.com/users/bjorng/followers","following_url":"https://api.github.com/users/bjorng/following{/other_user}","gists_url":"https://api.github.com/users/bjorng/gists{/gist_id}","starred_url":"https://api.github.com/users/bjorng/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bjorng/subscriptions","organizations_url":"https://api.github.com/users/bjorng/orgs","repos_url":"https://api.github.com/users/bjorng/repos","events_url":"https://api.github.com/users/bjorng/events{/privacy}","received_events_url":"https://api.github.com/users/bjorng/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"309079d194ea4c012a22c56f06cc9571a52b99b3","created_at":"2015-02-19T09:59:29Z","updated_at":"2015-02-19T09:59:29Z","body":"Please split this commit.\r\n\r\nSince mk_seg/0 is used only in new/0, I suggest that you eliminate mk_seg/0 and insert the code into new/0. That would be a separate commit.\r\n\r\nIf you are going to add erlang:make_tuple/2 to erl_bifs, it would make sense to also add erlang:make_tuple/3 and erlang:append_element/2 which are releated. That would be another commit.\r\n"}}