{"comment":{"url":"https://api.github.com/repos/WilliamParker/clara-rules/comments/20190338","html_url":"https://github.com/WilliamParker/clara-rules/commit/7c781284720e561a93e6f7011bcfc1704aa0f914#commitcomment-20190338","id":20190338,"user":{"login":"mrrodriguez","id":3483575,"avatar_url":"https://avatars.githubusercontent.com/u/3483575?v=3","gravatar_id":"","url":"https://api.github.com/users/mrrodriguez","html_url":"https://github.com/mrrodriguez","followers_url":"https://api.github.com/users/mrrodriguez/followers","following_url":"https://api.github.com/users/mrrodriguez/following{/other_user}","gists_url":"https://api.github.com/users/mrrodriguez/gists{/gist_id}","starred_url":"https://api.github.com/users/mrrodriguez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrrodriguez/subscriptions","organizations_url":"https://api.github.com/users/mrrodriguez/orgs","repos_url":"https://api.github.com/users/mrrodriguez/repos","events_url":"https://api.github.com/users/mrrodriguez/events{/privacy}","received_events_url":"https://api.github.com/users/mrrodriguez/received_events","type":"User","site_admin":false},"position":61,"line":839,"path":"src/main/clojure/clara/rules/memory.cljc","commit_id":"7c781284720e561a93e6f7011bcfc1704aa0f914","created_at":"2016-12-14T19:18:25Z","updated_at":"2016-12-14T19:18:25Z","body":"I don't really see the point of the use of unmodifiables here.  It doesn't cost a lot of time, sure, but it is unnecessary.  The state of working memory is mutable/update-in-place at this stage and all of the engine has to work with that situation.  Also, if we ended up later trying to pass one of these collections back into a \"transient\" working memory function, it would have to again be unwrapped to be able to be changed, which would be sneaky.\r\n\r\n"}}
{"actor":{"display_login":"mrrodriguez"}}