{"action":"created","issue":{"url":"https://api.github.com/repos/Project-OSRM/osrm-backend/issues/1878","labels_url":"https://api.github.com/repos/Project-OSRM/osrm-backend/issues/1878/labels{/name}","comments_url":"https://api.github.com/repos/Project-OSRM/osrm-backend/issues/1878/comments","events_url":"https://api.github.com/repos/Project-OSRM/osrm-backend/issues/1878/events","html_url":"https://github.com/Project-OSRM/osrm-backend/pull/1878","id":125943390,"number":1878,"title":"Inline mercator transformations; no need for separate translation unit","user":{"login":"daniel-j-h","id":527241,"avatar_url":"https://avatars.githubusercontent.com/u/527241?v=3","gravatar_id":"","url":"https://api.github.com/users/daniel-j-h","html_url":"https://github.com/daniel-j-h","followers_url":"https://api.github.com/users/daniel-j-h/followers","following_url":"https://api.github.com/users/daniel-j-h/following{/other_user}","gists_url":"https://api.github.com/users/daniel-j-h/gists{/gist_id}","starred_url":"https://api.github.com/users/daniel-j-h/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daniel-j-h/subscriptions","organizations_url":"https://api.github.com/users/daniel-j-h/orgs","repos_url":"https://api.github.com/users/daniel-j-h/repos","events_url":"https://api.github.com/users/daniel-j-h/events{/privacy}","received_events_url":"https://api.github.com/users/daniel-j-h/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-01-11T13:25:40Z","updated_at":"2016-01-11T16:56:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Project-OSRM/osrm-backend/pulls/1878","html_url":"https://github.com/Project-OSRM/osrm-backend/pull/1878","diff_url":"https://github.com/Project-OSRM/osrm-backend/pull/1878.diff","patch_url":"https://github.com/Project-OSRM/osrm-backend/pull/1878.patch"},"body":"Does what it says: no need for mercator to be a struct with two static functions, a namespace suffices (looks like we missed that in the other PR). No need for a header / impl. split for these one liners; let the optimizer see the body. Oh, and lat2y -> latToY, y2lat -> yToLat."},"comment":{"url":"https://api.github.com/repos/Project-OSRM/osrm-backend/issues/comments/170617093","html_url":"https://github.com/Project-OSRM/osrm-backend/pull/1878#issuecomment-170617093","issue_url":"https://api.github.com/repos/Project-OSRM/osrm-backend/issues/1878","id":170617093,"user":{"login":"TheMarex","id":606968,"avatar_url":"https://avatars.githubusercontent.com/u/606968?v=3","gravatar_id":"","url":"https://api.github.com/users/TheMarex","html_url":"https://github.com/TheMarex","followers_url":"https://api.github.com/users/TheMarex/followers","following_url":"https://api.github.com/users/TheMarex/following{/other_user}","gists_url":"https://api.github.com/users/TheMarex/gists{/gist_id}","starred_url":"https://api.github.com/users/TheMarex/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TheMarex/subscriptions","organizations_url":"https://api.github.com/users/TheMarex/orgs","repos_url":"https://api.github.com/users/TheMarex/repos","events_url":"https://api.github.com/users/TheMarex/events{/privacy}","received_events_url":"https://api.github.com/users/TheMarex/received_events","type":"User","site_admin":false},"created_at":"2016-01-11T16:56:38Z","updated_at":"2016-01-11T16:56:38Z","body":":+1: Looks good to me."}}