{"comment":{"url":"https://api.github.com/repos/testn/testng/comments/14530017","html_url":"https://github.com/testn/testng/commit/fa6485abed838702afdb0dd411369f34ad523e76#commitcomment-14530017","id":14530017,"user":{"login":"testn","id":445123,"avatar_url":"https://avatars.githubusercontent.com/u/445123?v=3","gravatar_id":"","url":"https://api.github.com/users/testn","html_url":"https://github.com/testn","followers_url":"https://api.github.com/users/testn/followers","following_url":"https://api.github.com/users/testn/following{/other_user}","gists_url":"https://api.github.com/users/testn/gists{/gist_id}","starred_url":"https://api.github.com/users/testn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/testn/subscriptions","organizations_url":"https://api.github.com/users/testn/orgs","repos_url":"https://api.github.com/users/testn/repos","events_url":"https://api.github.com/users/testn/events{/privacy}","received_events_url":"https://api.github.com/users/testn/received_events","type":"User","site_admin":false},"position":13,"line":44,"path":"src/main/java/org/testng/internal/Tarjan.java","commit_id":"fa6485abed838702afdb0dd411369f34ad523e76","created_at":"2015-11-20T15:48:51Z","updated_at":"2015-11-20T15:48:51Z","body":"The existing code was pretty much incorrect. It tried to compare Integer to Integer which would work fine for a small number of node due to Integer cache.\r\n\r\nThis simple test fails.\r\n```\r\n  @Test\r\n  public void testTest() {\r\n    Map<Integer, Integer> map = Maps.newHashMap();\r\n    Map<Integer, Integer> map2 = Maps.newHashMap();\r\n    map.put(1, 10000);\r\n    map2.put(1, 10000);\r\n    assertTrue(map.get(1)== map2.get(1));\r\n  }\r\n```"}}