I'd suggest culling these functions, but while we're here - arrays which don't change should always be constant - that way they can go in the read-only data section which can be shared among programs using the libxapian shared library (rather than using different memory in each process).

But in fact you can write this whole function body as simply:

```
    return strchr("NBSL", c) != NULL;
```