kbobyrev added inline comments.
================ Comment at: clang-tools-extra/clangd/index/dex/Iterator.cpp:181 + const auto HighestID = peek(); + DocID ChildrenToAdvance = 0; + while ((ChildrenToAdvance++ < Children.size()) && !reachedEnd() && ---------------- sammccall wrote: > I can't follow this one - why is the outer loop needed? > > (It looks like you're using DocID as integer again, but I can't tell why the > variable is needed at all) An artifact from min-heap implementation. https://reviews.llvm.org/D49546 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
