{"comment":{"url":"https://api.github.com/repos/JetBrains/kotlin/comments/14585137","html_url":"https://github.com/JetBrains/kotlin/commit/db509c8547357f787dff18d3af570b310867e19c#commitcomment-14585137","id":14585137,"user":{"login":"ligee","id":329565,"avatar_url":"https://avatars.githubusercontent.com/u/329565?v=3","gravatar_id":"","url":"https://api.github.com/users/ligee","html_url":"https://github.com/ligee","followers_url":"https://api.github.com/users/ligee/followers","following_url":"https://api.github.com/users/ligee/following{/other_user}","gists_url":"https://api.github.com/users/ligee/gists{/gist_id}","starred_url":"https://api.github.com/users/ligee/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ligee/subscriptions","organizations_url":"https://api.github.com/users/ligee/orgs","repos_url":"https://api.github.com/users/ligee/repos","events_url":"https://api.github.com/users/ligee/events{/privacy}","received_events_url":"https://api.github.com/users/ligee/received_events","type":"User","site_admin":false},"position":29,"line":93,"path":"compiler/rmi/rmi-server/src/org/jetbrains/kotlin/rmi/service/CompileServiceImpl.kt","commit_id":"db509c8547357f787dff18d3af570b310867e19c","created_at":"2015-11-24T10:52:02Z","updated_at":"2015-11-24T10:52:02Z","body":"I don't like the fact that state is modified in too many places in non-trivial way in this class. Makes it difficult to reason about. So I'd try to make it simpler e.g. by encapsulation state ops and organizing it differently. This is a bookmark for future considerations."}}