[PATCH] D73451: [clangd] Update the include mappings for std symbols.

Mon Jan 27 03:29:44 PST 2020

hokein added a comment.

In D73451#1841510 <https://reviews.llvm.org/D73451#1841510>, @kadircet wrote:

> we seem to be missing some symbols now, like `polymorphic_allocator` `random_shuffle` `gets` etc.

yeap, two majoir reasons:

- some symbols <https://en.cppreference.com/w/cpp/symbol_index/zombie_names> are deprecated/removed in new C++ standard, e.g. random_shuffle is removed in C++17
- missing information in cppreference page, e.g. there is no header section for `polymorphic_allocator` in the offline cppreference page :(

Regarding those deprecated symbols, we could handle them according to  which C++ standard we are using, as we have enough information from cppreferences <https://en.cppreference.com/w/cpp/symbol_index/zombie_names>, but it needs some work, and is low priority.

> and there seems to be a functional change to the parser. Are they intentional, if so why?

The change should be NFC, it just makes the parser more robust on invalid links, the new version of the cppreference have invalid links on the symbol index page.
updated the code to make it clearer.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73451/new/

https://reviews.llvm.org/D73451