search

mpetri / FM-Index

FM-Index full-text index implementation using RRR Wavelet trees (libcds) and fast suffix sorting (libdivsufsort) including experimental results.
GNU General Public License v3.0
98 stars 25 forks source link

Build frees input string memory #2

Open usamec opened 9 years ago

usamec commented 9 years ago

This line should not be here: https://github.com/mpetri/FM-Index/blob/master/FM.cpp#L127

I believe, that in C++ most people believe in RAII principle and do not expect FM index to free input string memory.

mpetri commented 9 years ago

you are right. I'm not really maintaining this anymore. You should probably use the sdsl-lite library (https://github.com/simongog/sdsl-lite) which contains many more succinct text indexes besides the FM-index and uses proper c++11 coding practices (at least for the most part).