The documentation's example for Comparator is buggy

[GoogleCodeExporter]

Hey!

I tried following the documentation for creating comparators and the line:

virtual const char* Name() { return "TwoPartComparator"; }

is wrong. It has to be:

virtual const char* Name() const { return "TwoPartComparator"; } 

instead (with a _const_ after Name()).

Thinking the documentation is perfect, I spent a long time trying to figure out 
what was wrong and figured at the end that the code was wrong to begin with. 
It's minor but I guess it would help fixing it.

Many thanks for open sourcing leveldb!
Amir

Original issue reported on code.google.com by amirhki...@gmail.com on 10 Aug 2011 at 7:37

[GoogleCodeExporter]

Thanks Amir!

Original comment by ga...@google.com on 10 Aug 2011 at 9:10

• Changed state: Accepted
• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

This is fixed in r42. Please verify.

Original comment by ga...@google.com on 16 Aug 2011 at 1:46

• Changed state: Fixed
• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

LGTM

Original comment by amirhki...@gmail.com on 16 Aug 2011 at 1:50

• Added labels: ****
• Removed labels: ****