{"action":"opened","issue":{"url":"https://api.github.com/repos/mrkkrp/text-metrics/issues/1","repository_url":"https://api.github.com/repos/mrkkrp/text-metrics","labels_url":"https://api.github.com/repos/mrkkrp/text-metrics/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/mrkkrp/text-metrics/issues/1/comments","events_url":"https://api.github.com/repos/mrkkrp/text-metrics/issues/1/events","html_url":"https://github.com/mrkkrp/text-metrics/issues/1","id":175074910,"number":1,"title":"Consider using alloca for memory allocation","user":{"login":"mrkkrp","id":8165792,"avatar_url":"https://avatars.githubusercontent.com/u/8165792?v=3","gravatar_id":"","url":"https://api.github.com/users/mrkkrp","html_url":"https://github.com/mrkkrp","followers_url":"https://api.github.com/users/mrkkrp/followers","following_url":"https://api.github.com/users/mrkkrp/following{/other_user}","gists_url":"https://api.github.com/users/mrkkrp/gists{/gist_id}","starred_url":"https://api.github.com/users/mrkkrp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrkkrp/subscriptions","organizations_url":"https://api.github.com/users/mrkkrp/orgs","repos_url":"https://api.github.com/users/mrkkrp/repos","events_url":"https://api.github.com/users/mrkkrp/events{/privacy}","received_events_url":"https://api.github.com/users/mrkkrp/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"mrkkrp","id":8165792,"avatar_url":"https://avatars.githubusercontent.com/u/8165792?v=3","gravatar_id":"","url":"https://api.github.com/users/mrkkrp","html_url":"https://github.com/mrkkrp","followers_url":"https://api.github.com/users/mrkkrp/followers","following_url":"https://api.github.com/users/mrkkrp/following{/other_user}","gists_url":"https://api.github.com/users/mrkkrp/gists{/gist_id}","starred_url":"https://api.github.com/users/mrkkrp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrkkrp/subscriptions","organizations_url":"https://api.github.com/users/mrkkrp/orgs","repos_url":"https://api.github.com/users/mrkkrp/repos","events_url":"https://api.github.com/users/mrkkrp/events{/privacy}","received_events_url":"https://api.github.com/users/mrkkrp/received_events","type":"User","site_admin":false},"assignees":[{"login":"mrkkrp","id":8165792,"avatar_url":"https://avatars.githubusercontent.com/u/8165792?v=3","gravatar_id":"","url":"https://api.github.com/users/mrkkrp","html_url":"https://github.com/mrkkrp","followers_url":"https://api.github.com/users/mrkkrp/followers","following_url":"https://api.github.com/users/mrkkrp/following{/other_user}","gists_url":"https://api.github.com/users/mrkkrp/gists{/gist_id}","starred_url":"https://api.github.com/users/mrkkrp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrkkrp/subscriptions","organizations_url":"https://api.github.com/users/mrkkrp/orgs","repos_url":"https://api.github.com/users/mrkkrp/repos","events_url":"https://api.github.com/users/mrkkrp/events{/privacy}","received_events_url":"https://api.github.com/users/mrkkrp/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/mrkkrp/text-metrics/milestones/1","html_url":"https://github.com/mrkkrp/text-metrics/milestones/0.1.1","labels_url":"https://api.github.com/repos/mrkkrp/text-metrics/milestones/1/labels","id":1984270,"number":1,"title":"0.1.1","description":null,"creator":{"login":"mrkkrp","id":8165792,"avatar_url":"https://avatars.githubusercontent.com/u/8165792?v=3","gravatar_id":"","url":"https://api.github.com/users/mrkkrp","html_url":"https://github.com/mrkkrp","followers_url":"https://api.github.com/users/mrkkrp/followers","following_url":"https://api.github.com/users/mrkkrp/following{/other_user}","gists_url":"https://api.github.com/users/mrkkrp/gists{/gist_id}","starred_url":"https://api.github.com/users/mrkkrp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrkkrp/subscriptions","organizations_url":"https://api.github.com/users/mrkkrp/orgs","repos_url":"https://api.github.com/users/mrkkrp/repos","events_url":"https://api.github.com/users/mrkkrp/events{/privacy}","received_events_url":"https://api.github.com/users/mrkkrp/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":0,"state":"open","created_at":"2016-09-05T13:46:00Z","updated_at":"2016-09-05T13:46:03Z","due_on":null,"closed_at":null},"comments":0,"created_at":"2016-09-05T13:46:03Z","updated_at":"2016-09-05T13:46:03Z","closed_at":null,"body":"As @alexbiehl points out, `alloca` may be a better choice for our use case since it allocates the memory on stack, thus it's much cheaper an operation than allocating memory from heap and then releasing it. I measured that this will make `levenshtein` function 25% faster for example. If we decide to use `alloca`, we should note in the docs that memory for distance vectors is allocated on stack, so very long input values may lead to stack overflows (should not be a problem since the functions are typically used with not very long input values)."}}
{"actor":{"display_login":"mrkkrp"}}