{"action":"created","issue":{"url":"https://api.github.com/repos/michaeljones/breathe/issues/191","labels_url":"https://api.github.com/repos/michaeljones/breathe/issues/191/labels{/name}","comments_url":"https://api.github.com/repos/michaeljones/breathe/issues/191/comments","events_url":"https://api.github.com/repos/michaeljones/breathe/issues/191/events","html_url":"https://github.com/michaeljones/breathe/pull/191","id":80993456,"number":191,"title":"Remove template arguments from the class name in the member function","user":{"login":"vitaut","id":576385,"avatar_url":"https://avatars.githubusercontent.com/u/576385?v=3","gravatar_id":"","url":"https://api.github.com/users/vitaut","html_url":"https://github.com/vitaut","followers_url":"https://api.github.com/users/vitaut/followers","following_url":"https://api.github.com/users/vitaut/following{/other_user}","gists_url":"https://api.github.com/users/vitaut/gists{/gist_id}","starred_url":"https://api.github.com/users/vitaut/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vitaut/subscriptions","organizations_url":"https://api.github.com/users/vitaut/orgs","repos_url":"https://api.github.com/users/vitaut/repos","events_url":"https://api.github.com/users/vitaut/events{/privacy}","received_events_url":"https://api.github.com/users/vitaut/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-05-26T14:41:15Z","updated_at":"2015-05-27T14:17:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/michaeljones/breathe/pulls/191","html_url":"https://github.com/michaeljones/breathe/pull/191","diff_url":"https://github.com/michaeljones/breathe/pull/191.diff","patch_url":"https://github.com/michaeljones/breathe/pull/191.patch"},"body":"The proposed PR removes template arguments from the class name in the member function. Otherwise links to nested classes (in parameter or return types) defined in the same template are not generated correctly, because the latter don't include template arguments.\r\n\r\nFor example, `void C<T>::f(D)` is converted to `void C::f(D)` in\r\n\r\n```c++\r\n/** A class */\r\ntemplate <typename T>\r\nclass C {\r\n public:\r\n  /** An inner class. */\r\n  class D {};\r\n\r\n  /** A method. */\r\n  void f(D);\r\n};\r\n```\r\n\r\nFixes #179.\r\n\r\n@michaeljones, if you merged #189 first, I could also add a unit test for this."},"comment":{"url":"https://api.github.com/repos/michaeljones/breathe/issues/comments/105927885","html_url":"https://github.com/michaeljones/breathe/pull/191#issuecomment-105927885","issue_url":"https://api.github.com/repos/michaeljones/breathe/issues/191","id":105927885,"user":{"login":"vitaut","id":576385,"avatar_url":"https://avatars.githubusercontent.com/u/576385?v=3","gravatar_id":"","url":"https://api.github.com/users/vitaut","html_url":"https://github.com/vitaut","followers_url":"https://api.github.com/users/vitaut/followers","following_url":"https://api.github.com/users/vitaut/following{/other_user}","gists_url":"https://api.github.com/users/vitaut/gists{/gist_id}","starred_url":"https://api.github.com/users/vitaut/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vitaut/subscriptions","organizations_url":"https://api.github.com/users/vitaut/orgs","repos_url":"https://api.github.com/users/vitaut/repos","events_url":"https://api.github.com/users/vitaut/events{/privacy}","received_events_url":"https://api.github.com/users/vitaut/received_events","type":"User","site_admin":false},"created_at":"2015-05-27T14:17:08Z","updated_at":"2015-05-27T14:17:08Z","body":"Thanks for merging the unit testing branch. I've moved the code that removes template arguments to a separate function and added a unit test for it."}}