{"comment":{"url":"https://api.github.com/repos/kylestev/jvm.js/comments/14145582","html_url":"https://github.com/kylestev/jvm.js/commit/4a2e945b24096629e76544c372221ab59b4b2dfb#commitcomment-14145582","id":14145582,"user":{"login":"kylestev","id":819571,"avatar_url":"https://avatars.githubusercontent.com/u/819571?v=3","gravatar_id":"","url":"https://api.github.com/users/kylestev","html_url":"https://github.com/kylestev","followers_url":"https://api.github.com/users/kylestev/followers","following_url":"https://api.github.com/users/kylestev/following{/other_user}","gists_url":"https://api.github.com/users/kylestev/gists{/gist_id}","starred_url":"https://api.github.com/users/kylestev/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kylestev/subscriptions","organizations_url":"https://api.github.com/users/kylestev/orgs","repos_url":"https://api.github.com/users/kylestev/repos","events_url":"https://api.github.com/users/kylestev/events{/privacy}","received_events_url":"https://api.github.com/users/kylestev/received_events","type":"User","site_admin":false},"position":4,"line":31,"path":"src/core/parsers/BytecodeInstructions.js","commit_id":"4a2e945b24096629e76544c372221ab59b4b2dfb","created_at":"2015-11-03T05:02:41Z","updated_at":"2015-11-03T05:02:41Z","body":"If you want to set this property, I'd recommend modifying `InstructionFactory` to accept it in `InstructionFactory#of` which would then pass it into the constructor of the instruction it will create. If you don't want to do that, at the very least, you'll need to provide getters/setters for the owner property of an instruction otherwise it is not clear that the property even exists when looking at the `AbstractInstruction` class."}}