This is an archive of the discontinued LLVM Phabricator instance.

[clangd] A code action to remove 'using namespace'
AbandonedPublic

Authored by ilya-biryukov on Jan 11 2019, 10:32 AM.

Details

Reviewers
sammccall
Summary

Only available in the source files to fit into the model of single-file
actions. Doing the same in headers would require more complicated
machinery, which is out of scope of code actions.

Event Timeline

ilya-biryukov created this revision.Jan 11 2019, 10:32 AM
ilya-biryukov planned changes to this revision.Jan 11 2019, 10:34 AM

The most complicated of the sample actions, requires considerable work and thorough testing before it can be landed.
Serves the purpose of illustrating how to write the two-stage actions.

  • Update after changes to parent revision
  • Rebase after parent change
  • Update to reflect changes in parent revision
  • Remove the header file
  • Move to the monorepo
  • Update license header
nridge added a subscriber: nridge.Oct 1 2019, 3:45 PM
ilya-biryukov abandoned this revision.Oct 16 2019, 5:58 AM

This was finalized and landed as D68562