forked from OSchip/llvm-project
766338ad7f
Summary: To implement incremental document syncing, we want to verify that the ranges provided by the front-end are valid. Currently, positionToOffset deals with invalid Positions by returning 0 or Code.size(), which are two valid offsets. Instead, return an llvm:Expected<size_t> with an error if the position is invalid. According to the LSP, if the character value exceeds the number of characters of the given line, it should default back to the end of the line. It makes sense in some contexts to have this behavior, and does not in other contexts. The AllowColumnsBeyondLineLength parameter allows to decide what to do in that case, default back to the end of the line, or return an error. Reviewers: ilya-biryukov Subscribers: klimek, ilya-biryukov, jkorous-apple, ioeric, cfe-commits Differential Revision: https://reviews.llvm.org/D44673 llvm-svn: 328100 |
||
|---|---|---|
| .. | ||
| change-namespace | ||
| clang-apply-replacements | ||
| clang-move | ||
| clang-query | ||
| clang-tidy | ||
| clangd | ||
| include/common | ||
| include-fixer | ||
| CMakeLists.txt | ||