{"action":"closed","issue":{"url":"https://api.github.com/repos/chrsmith/bwapi/issues/89","labels_url":"https://api.github.com/repos/chrsmith/bwapi/issues/89/labels{/name}","comments_url":"https://api.github.com/repos/chrsmith/bwapi/issues/89/comments","events_url":"https://api.github.com/repos/chrsmith/bwapi/issues/89/events","html_url":"https://github.com/chrsmith/bwapi/issues/89","id":58653573,"number":89,"title":"BWAPI crashes with an when loading any replay","user":{"login":"GoogleCodeExporter","id":9614759,"avatar_url":"https://avatars.githubusercontent.com/u/9614759?v=3","gravatar_id":"","url":"https://api.github.com/users/GoogleCodeExporter","html_url":"https://github.com/GoogleCodeExporter","followers_url":"https://api.github.com/users/GoogleCodeExporter/followers","following_url":"https://api.github.com/users/GoogleCodeExporter/following{/other_user}","gists_url":"https://api.github.com/users/GoogleCodeExporter/gists{/gist_id}","starred_url":"https://api.github.com/users/GoogleCodeExporter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GoogleCodeExporter/subscriptions","organizations_url":"https://api.github.com/users/GoogleCodeExporter/orgs","repos_url":"https://api.github.com/users/GoogleCodeExporter/repos","events_url":"https://api.github.com/users/GoogleCodeExporter/events{/privacy}","received_events_url":"https://api.github.com/users/GoogleCodeExporter/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/chrsmith/bwapi/labels/auto-migrated","name":"auto-migrated","color":"ededed"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-02-23T21:42:53Z","updated_at":"2015-02-23T21:42:53Z","closed_at":"2015-02-23T21:42:53Z","body":"```\nWhat steps will reproduce the problem?\n1. load any replays with the ExampleAI loaded(any script should do it though)\n2. See BW crash\n\nWhat is the expected output? What do you see instead?\nThe replay should to continue until completion.\n\nWhat version of the product are you using? On what operating system?\nWindows XPSP3, BWAPI_Beta2 and svn r1410\n\nPlease provide any additional information below.\nThis is arguably a duplicate of Issue79 but as that one is marked as\nwontfix and the bug still exists I wanted to increase the visibility.\n\nIt seems the problem is happening because GameImpl::BWAPIPlayer is not\ngetting set in GameImpl::onGameStart() and it is getting referenced in\nUnitImpl::isVisible() (line523).\nSo far I have changed my local version to have a check in the isVisible()\nmethod for a NULL Broodwar->self() but I think that the \"correct\" fix may\nbe to pick a random(1st?) player as self in the onGameStart() method.\n```\n\n-----\nOriginal issue reported on code.google.com by `iulia...@gmail.com` on 14 Nov 2009 at 8:59"}}