ioeric added inline comments.
================ Comment at: clangd/index/Index.h:27 + // The URI of the source file where a symbol occurs. + llvm::StringRef FileUri; // The 0-based offset to the first character of the symbol from the beginning ---------------- hokein wrote: > sammccall wrote: > > ioeric wrote: > > > sammccall wrote: > > > > nit: FileURI? > > > > (The other style is OK too, though I personally find it harder to read. > > > > But the class is `URI` and we should be consistent) > > > I tend to use `Uri` for URI strings and `URI`/`U` for URI objects. But > > > I'd be okay with either, if you prefer `URI`. > > I'm fine with either spelling, but not really with mixing the two. > > > > Is there some precedent for using different capitalization depending on the > > variable type? That seems... non obvious to me. > I'm +1 on `URI`, using all capital letters for initialisms can make the name > a bit easier to read. Let's be consistent using `URI`? I'll change to `URI`. Another reason why I avoided using URI as variable names was the name conflicts with the actual URI class... I always need to come up with a prefix... Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D42915 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits