This is an archive of the discontinued LLVM Phabricator instance.

[clangd] IncludeFixer: resolve Class in ns::Class::method();
Needs ReviewPublic

Authored by sammccall on Feb 26 2022, 7:22 AM.

Details

Reviewers
usaxena95
Summary

We distinguish by case, assuming a::b:: is a namespace and a::B:: is a class.
Crude, but be a significant improvement over assuming it's always a namespace.

(Motivating case from clangd: clangd::CommandMangler::detect()).

Diff Detail

Event Timeline

sammccall created this revision.Feb 26 2022, 7:22 AM
sammccall requested review of this revision.Feb 26 2022, 7:22 AM

Friendly ping :-)

Herald added a project: Restricted Project. · View Herald TranscriptMar 23 2022, 10:38 AM