{"action":"created","issue":{"url":"https://api.github.com/repos/chrsmith/leveldb/issues/28","labels_url":"https://api.github.com/repos/chrsmith/leveldb/issues/28/labels{/name}","comments_url":"https://api.github.com/repos/chrsmith/leveldb/issues/28/comments","events_url":"https://api.github.com/repos/chrsmith/leveldb/issues/28/events","html_url":"https://github.com/chrsmith/leveldb/issues/28","id":58996659,"number":28,"title":"The documentation's example for Comparator is buggy","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/leveldb/labels/auto-migrated","name":"auto-migrated","color":"ededed"},{"url":"https://api.github.com/repos/chrsmith/leveldb/labels/Priority-Medium","name":"Priority-Medium","color":"ededed"},{"url":"https://api.github.com/repos/chrsmith/leveldb/labels/Type-Defect","name":"Type-Defect","color":"ededed"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-02-26T00:36:06Z","updated_at":"2015-02-26T00:36:07Z","closed_at":"2015-02-26T00:36:07Z","body":"```\nHey!\n\nI tried following the documentation for creating comparators and the line:\n\nvirtual const char* Name() { return \"TwoPartComparator\"; }\n\nis wrong. It has to be:\n\nvirtual const char* Name() const { return \"TwoPartComparator\"; } \n\ninstead (with a _const_ after Name()).\n\nThinking the documentation is perfect, I spent a long time trying to figure out \nwhat was wrong and figured at the end that the code was wrong to begin with. \nIt's minor but I guess it would help fixing it.\n\nMany thanks for open sourcing leveldb!\nAmir\n\n\n```\n\n-----\nOriginal issue reported on code.google.com by `amirhki...@gmail.com` on 10 Aug 2011 at 7:37"},"comment":{"url":"https://api.github.com/repos/chrsmith/leveldb/issues/comments/76097827","html_url":"https://github.com/chrsmith/leveldb/issues/28#issuecomment-76097827","issue_url":"https://api.github.com/repos/chrsmith/leveldb/issues/28","id":76097827,"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-26T00:36:07Z","updated_at":"2015-02-26T00:36:07Z","body":"```\nThis is fixed in r42. Please verify.\n```\n\n-----\nOriginal comment by `ga...@google.com` on 16 Aug 2011 at 1:46\n* Changed state: **Fixed**"}}