{"comment":{"url":"https://api.github.com/repos/randombit/botan/comments/9203238","html_url":"https://github.com/randombit/botan/commit/a205bc8691df95b58e651fdb932d886f4f937cb9#commitcomment-9203238","id":9203238,"user":{"login":"webmaster128","id":2603011,"avatar_url":"https://avatars.githubusercontent.com/u/2603011?v=3","gravatar_id":"","url":"https://api.github.com/users/webmaster128","html_url":"https://github.com/webmaster128","followers_url":"https://api.github.com/users/webmaster128/followers","following_url":"https://api.github.com/users/webmaster128/following{/other_user}","gists_url":"https://api.github.com/users/webmaster128/gists{/gist_id}","starred_url":"https://api.github.com/users/webmaster128/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/webmaster128/subscriptions","organizations_url":"https://api.github.com/users/webmaster128/orgs","repos_url":"https://api.github.com/users/webmaster128/repos","events_url":"https://api.github.com/users/webmaster128/events{/privacy}","received_events_url":"https://api.github.com/users/webmaster128/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"a205bc8691df95b58e651fdb932d886f4f937cb9","created_at":"2015-01-08T15:17:56Z","updated_at":"2015-01-08T15:17:56Z","body":"LGTM. To avoid spaghetti code, I'd personally do the case distinction when setting `output_length`, because the calculation of `output_length` is the core of the problem. Optimizing runtime for input_length should not be useful here. "}}