Re: [clang] cf3421d - [Format] Fix a warning

2024-01-18 Thread Kazu Hirata via cfe-commits
Good point! Thanks! On Wed, Jan 17, 2024 at 3:34 PM David Blaikie wrote: > In general, if the lambda is only used within its scope, I'd suggest > switching to a default ref capture "[&]" to reduce maintenance burden of > keeping the capture list up to date. > > On Thu, Jan 11, 2024 at 12:09 PM

Re: [clang] cf3421d - [Format] Fix a warning

2024-01-17 Thread David Blaikie via cfe-commits
In general, if the lambda is only used within its scope, I'd suggest switching to a default ref capture "[&]" to reduce maintenance burden of keeping the capture list up to date. On Thu, Jan 11, 2024 at 12:09 PM Kazu Hirata via cfe-commits < cfe-commits@lists.llvm.org> wrote: > > Author: Kazu

[clang] cf3421d - [Format] Fix a warning

2024-01-11 Thread Kazu Hirata via cfe-commits
Author: Kazu Hirata Date: 2024-01-11T12:09:01-08:00 New Revision: cf3421de587d7c947e8f6b5c754393f85a395747 URL: https://github.com/llvm/llvm-project/commit/cf3421de587d7c947e8f6b5c754393f85a395747 DIFF: https://github.com/llvm/llvm-project/commit/cf3421de587d7c947e8f6b5c754393f85a395747.diff