[PATCH] D68245: [Clangd] ExtractFunction: Don't extract body of enclosing function.

2019-09-30 Thread Shaurya Gupta via Phabricator via cfe-commits
SureYeaah created this revision. SureYeaah added reviewers: sammccall, kadircet. Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. This patch disable extraction of the body of the enclosing function. `void f() [[{}]]` Extr

[PATCH] D68245: [Clangd] ExtractFunction: Don't extract body of enclosing function.

2019-10-01 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. LGTM, thanks Comment at: clang-tools-extra/clangd/refactor/tweaks/ExtractFunction.cpp:231 + if (const Stmt *S = Child->ASTNode.get()) +if (EnclosingFunc->hasBody() &

[PATCH] D68245: [Clangd] ExtractFunction: Don't extract body of enclosing function.

2019-10-02 Thread Shaurya Gupta via Phabricator via cfe-commits
SureYeaah updated this revision to Diff 222815. SureYeaah marked an inline comment as done. SureYeaah added a comment. Added assert and removed fixme Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68245/new/ https://reviews.llvm.org/D68245 Files:

[PATCH] D68245: [Clangd] ExtractFunction: Don't extract body of enclosing function.

2019-10-02 Thread Shaurya Gupta via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL373472: [Clangd] ExtractFunction: Don't extract body of enclosing function. (authored by SureYeaah, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to