{"comment":{"url":"https://api.github.com/repos/CCI-MIT/XCoLab/comments/16686238","html_url":"https://github.com/CCI-MIT/XCoLab/commit/d84011ea51a6c3df8cb40f79b70165ee7f9b496f#commitcomment-16686238","id":16686238,"user":{"login":"jobachh","id":6458867,"avatar_url":"https://avatars.githubusercontent.com/u/6458867?v=3","gravatar_id":"","url":"https://api.github.com/users/jobachh","html_url":"https://github.com/jobachh","followers_url":"https://api.github.com/users/jobachh/followers","following_url":"https://api.github.com/users/jobachh/following{/other_user}","gists_url":"https://api.github.com/users/jobachh/gists{/gist_id}","starred_url":"https://api.github.com/users/jobachh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jobachh/subscriptions","organizations_url":"https://api.github.com/users/jobachh/orgs","repos_url":"https://api.github.com/users/jobachh/repos","events_url":"https://api.github.com/users/jobachh/events{/privacy}","received_events_url":"https://api.github.com/users/jobachh/received_events","type":"User","site_admin":false},"position":32,"line":32,"path":"services/plansProposalsFacade/plansProposalsFacade-portlet-service/src/main/java/org/xcolab/helpers/Tuple.java","commit_id":"d84011ea51a6c3df8cb40f79b70165ee7f9b496f","created_at":"2016-03-14T17:07:15Z","updated_at":"2016-03-14T17:07:15Z","body":"You are checking the same condition on two different objects but you do it in a completely different way - this is very confusing to read. Simplify to:\r\n`return left.equals(tuple.left) && right.equals(tuple.right);`"}}