Page MenuHomePhabricator

[clangd] Drop returntype/type when hovering over type-ish names
ClosedPublic

Authored by kadircet on Tue, Jan 21, 7:50 AM.

Details

Summary

Some names, e.g. constructor/destructor/conversions, already contain
the type info, no need to duplicate them in the hoverinfo.

Fixes https://github.com/clangd/clangd/issues/252

Diff Detail

Event Timeline

kadircet created this revision.Tue, Jan 21, 7:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptTue, Jan 21, 7:50 AM

Unit tests: pass. 62057 tests passed, 0 failed and 784 were skipped.

clang-tidy: unknown.

clang-format: pass.

Build artifacts: diff.json, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

sammccall accepted this revision.Tue, Jan 21, 8:04 AM
This revision is now accepted and ready to land.Tue, Jan 21, 8:04 AM
This revision was automatically updated to reflect the committed changes.