[PATCH] D52084: [clangd] NFC: Update documentation of Iterator's dump format

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 17 00:44:07 PDT 2018


ioeric accepted this revision.
ioeric added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang-tools-extra/clangd/index/dex/Iterator.h:96
   /// ChildN is N-th iterator child. Raw iterators over PostingList are
-  /// represented as "[ID1, ID2, ..., {IDN}, ... END]" where IDN is N-th
-  /// PostingList entry and the element which is pointed to by the PostingList
-  /// iterator is enclosed in {} braces.
+  /// represented as "[ID]" where ID is the element under iterator cursor.
   friend llvm::raw_ostream &operator<<(llvm::raw_ostream &OS,
----------------
I'd probably print `... [ID] ...` to indicate that there can be other elements in the list.


https://reviews.llvm.org/D52084





More information about the cfe-commits mailing list