[PATCH] D40481: [libclang] Fix cursors for arguments of Subscript and Call operators

Nikolai Kosjar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 27 03:01:05 PST 2017


nik marked 4 inline comments as done.
nik added inline comments.


================
Comment at: tools/libclang/CIndex.cpp:6888
+      const SourceLocation fixedEnd =
+          RefNameRange.getEnd().getLocWithOffset(-1);
+      RefNameRange = SourceRange(RefNameRange.getBegin(), fixedEnd);
----------------
yvvan wrote:
> 4 spaces indentation instead of 2
That's not me, but clang-format, so I guess it's fine as is.


================
Comment at: tools/libclang/CIndex.cpp:6892
+      const RangeComparisonResult ComparisonResult =
+          LocationCompare(SrcMgr, TokenLocation, RefNameRange);
+
----------------
yvvan wrote:
> same
That's not me, but clang-format, so I guess it's fine as is.


https://reviews.llvm.org/D40481





More information about the cfe-commits mailing list