{"action":"created","issue":{"url":"https://api.github.com/repos/chrsmith/bwapi/issues/95","labels_url":"https://api.github.com/repos/chrsmith/bwapi/issues/95/labels{/name}","comments_url":"https://api.github.com/repos/chrsmith/bwapi/issues/95/comments","events_url":"https://api.github.com/repos/chrsmith/bwapi/issues/95/events","html_url":"https://github.com/chrsmith/bwapi/issues/95","id":58483746,"number":95,"title":"Minimize overhead in draw functions","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"},{"url":"https://api.github.com/repos/chrsmith/bwapi/labels/Priority-Medium","name":"Priority-Medium","color":"ededed"},{"url":"https://api.github.com/repos/chrsmith/bwapi/labels/Type-Enhancement","name":"Type-Enhancement","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-02-22T00:52:38Z","updated_at":"2015-02-22T00:52:38Z","closed_at":null,"body":"```\nCurrently the drawX methods have an additonal CoordinateType parameter \nwith an appropiate switch behind the scene. This overhead adds up when \ndrawing multi layered full-map debug info.\n\nSince the CoordinateType parameter in 99.9% of the cases is not changed \ndynamically, I suggest either:\n- 3 different methods e.g.\n  drawScreenLine(from_x, from_y, ...);\n  drawMapLine(from, from_y, ...);\n  drawMouseLine(from, from_y, ...);\n- a template parameter e.g.\n  drawLine<CoordinateType::Map>(from, from_y, ...);\n- a separate coordinate conversion to use only when necessary\n  drawLine(convertMouseToMap(from_coord), ...);\n\nThe last one is a kinda clumsy solution...\n```\n\n-----\nOriginal issue reported on code.google.com by `goo...@teabix.com` on 15 Nov 2009 at 6:54"},"comment":{"url":"https://api.github.com/repos/chrsmith/bwapi/issues/comments/75408901","html_url":"https://github.com/chrsmith/bwapi/issues/95#issuecomment-75408901","issue_url":"https://api.github.com/repos/chrsmith/bwapi/issues/95","id":75408901,"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},"created_at":"2015-02-22T00:52:38Z","updated_at":"2015-02-22T00:52:38Z","body":"\n\n-----\nOriginal comment by `AHeinerm` on 15 Nov 2009 at 8:01\n* Changed state: **Fixed**"}}