This is an archive of the discontinued LLVM Phabricator instance.

[docs] Update PostingList string representation format
ClosedPublic

Authored by kbobyrev on Sep 26 2018, 5:44 AM.

Details

Summary

Because PostingList objects are compressed, it is now impossible to see elements other than the current one and the documentation doesn't match implementation anymore.

Diff Detail

Repository
rL LLVM

Event Timeline

kbobyrev created this revision.Sep 26 2018, 5:44 AM
ioeric added inline comments.Sep 26 2018, 7:54 AM
clang-tools-extra/clangd/index/dex/Iterator.h
97 ↗(On Diff #167114)

nit: [(...)? (IDN | END) (...)?] is accurate but can be really confusing... I think it's okay to be a bit inaccurate with just [ ... CurID ... ].

ioeric accepted this revision.Sep 26 2018, 7:54 AM
This revision is now accepted and ready to land.Sep 26 2018, 7:54 AM
kbobyrev updated this revision to Diff 167138.Sep 26 2018, 7:58 AM

Simplify the documentation format.

This revision was automatically updated to reflect the committed changes.