This says "start exclusive, end inclusive", but code:

```
        if (CodeMirror.cmpPos(pos, start) < 0) {
            return pos;
        }
        if (CodeMirror.cmpPos(pos, end) <= 0) {
            return CodeMirror.changeEnd(change);
        }
```

is "start inclusive, end exclusive", right?