sdsl

sdsl copied to clipboard

Problems with install.sh (-Wnarrowing)

1

Dear all, I'm having a problem when building sdsl by executing install.sh. There is a compilation error due -Wnarrowing The compilation log is attached: /home/danielsaad/Desktop/sdsl-master/lib/bwt_construct.cpp: In function ‘bool sdsl::construct_bwt(sdsl::tMSS&, const...

danielsaad

Change Licence

1

...to LGPL

simongog

tb38

assert() in access operations

1

it would be nice to have assert() statements in all access operators and for example rank() / select() function as it is quite easy to have an "off by one"...

mpetri

libdivsufsort should use int_vector<>

1

now that libdivsufsort is included in the library it would be very nice to modify it to use the sdsl int vector to enable bit compression among other things.

mpetri

Does this lib only can be run on a 64-bits system?

4

1) I find the uint128_t be defined as below: typedef unsigned int uint128_t **attribute**((mode(TI))); It can't be complier on my box,I google it for that,on 32-bits system the TI should...

feixeyes

file helper class should be in utils.hpp

1

the file::read_text function is used throughout the library to read raw text from disk. it should therefore be in the utils:: class/namespace and not in the testutils.hpp file where nobody...

mpetri

The sampling parameter in the csa classes (csa_wt, csa_sada, csa_uncompressed) are template parameter. The original reason for this design decision was that in operator[] and operator() there are modulo and...

simongog