sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.

LG, with

- consider reverting the bitpacking stuff
- comment about utf-16
- clang-format :)

Comment at: clangd/index/SymbolCollector.cpp:202
+    SymbolLocation::Position Pos;
+    // Position is 0-based while source location is 1-based.
+    Pos.Line = SM.getLineNumber(FileId, Offset) - 1;
nit: SourceManager is 1-based (or returns 1-based data here).
SourceLocation uses 0-based offsets, not 1-based line/column.

  rCTE Clang Tools Extra

cfe-commits mailing list

Reply via email to