Rebased!

I'm -0 on this. Even if this is merged, how could we prevent implicit captures in the future?

@promag What are the best arguments in favour of implicit captures? :-)

Tend towards NACK. Sorry to rant, but I'm starting to feel quite strongly about this:

If you want to make changes like this, the way to do so is:

• create a PR to update the coding guidelines with your motivation on how this avoids bugs (as you did in the OP), and that your point is strong enough for this to be merged
• if it is merged, make sure it is applied to new code, in reviews
• then maybe, in the longer term, if people agree that this is important, do a PR that updates old occurrences

Please don't create a PR out of the blue that changes some formerly unheard-of thing in the entire codebase. This creates review work, extra risk, and all massive idempotent code changes all over the code make it harder to keep track of what is actually changing.

@promag Thanks for reviewing. I've now updated the PR. Implicit captures are now kept for trivial lambdas with only one capture. Explicit captures are used when the capture count is two or above. What do you think? Please re-review :-)

@laanwj Makes sense! Closing this PR