Re: [clang-tools-extra] r321192 - [clangd] Remove an unused lambda capture.

2017-12-25 Thread David Blaikie via cfe-commits
Generally I'd encourage you/anyone to use [&] for any lambda that doesn't escape its scope - avoids any issues like this & doesn't really hinder readability imho. On Wed, Dec 20, 2017 at 9:23 AM Eric Liu via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: ioeric > Date: Wed Dec 20 09:2

[clang-tools-extra] r321192 - [clangd] Remove an unused lambda capture.

2017-12-20 Thread Eric Liu via cfe-commits
Author: ioeric Date: Wed Dec 20 09:22:56 2017 New Revision: 321192 URL: http://llvm.org/viewvc/llvm-project?rev=321192&view=rev Log: [clangd] Remove an unused lambda capture. Modified: clang-tools-extra/trunk/unittests/clangd/Annotations.cpp Modified: clang-tools-extra/trunk/unittests/clangd