This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Add flag to control #import include insertions
ClosedPublic

Authored by dgoldman on Dec 6 2022, 11:26 AM.

Details

Summary

This will be disabled by default, hopefully we can enable for the next
major release.

Diff Detail

Event Timeline

dgoldman created this revision.Dec 6 2022, 11:26 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 6 2022, 11:26 AM
Herald added a subscriber: arphaman. · View Herald Transcript
dgoldman requested review of this revision.Dec 6 2022, 11:26 AM
kadircet accepted this revision.Jan 4 2023, 11:58 PM
kadircet added inline comments.
clang-tools-extra/clangd/tool/ClangdMain.cpp
271

... accepting code completions or fixing includes in Objective-C code.

272

s/false/CodeCompleteOptions().ImportInsertions/

This revision is now accepted and ready to land.Jan 4 2023, 11:58 PM
dgoldman updated this revision to Diff 486700.Jan 5 2023, 3:43 PM
dgoldman marked 2 inline comments as done.

Update comment + default value

This revision was automatically updated to reflect the committed changes.