[libcxx-commits] [libcxx] [libc++] Diagnose unused variables of container types (PR #203084)

[Yuxuan Chen via libcxx-commits]

yuxuanchen1997 wrote:

> Not if I don't think it's beneficial. IMO the macro is worse to read and improves nothing to make up for that. If I understand why you want it and think it's a reasonable ask I may be persuaded, but (especially the first part) isn't there so far.

Risk of repeating myself: I don't want to start diagnosing unused containers overnight and want to opt out of this change temporarily, I want a simpler change than having to apply a reverse patch to this PR. The PR as is today provides no option to revert to the previous behaviour. The benefit here being giving people the time to test this change and migrate. 

https://github.com/llvm/llvm-project/pull/203084