Segmentation fault in lcp for the terminal symbol alone

[hynekcer]

I'm testing the package on random data:

>>> sequence = '$'
>>> sa = pysais.sais(sequence)
>>> lcp, lcp_lm, lcp_mr = pysais.lcp(sequence, sa)
Segmentation fault (core dumped)

(I observed segmentation faults also on normal data sometimes if I omitted the terminal symbol. But I understand that you strictly require it.)

[AlexeyG]

The corner case of sequence = '$' should be easily fixable with an if statement. Same goes for checking the presence of the terminal symbol. But the way I see it, these checks of input data just eat up CPU cycles. And the only reason why I'm using the SAIS C module is to gain the speed that would be unattainable with a pure python implementation.

You're very welcome to add the checks and I'll merge it.