- `fun` was used for variable/field names in a number of places.
These have been replaced by similar or more descriptive names,
to avoid name shadowing, which is not allowed in the GCC-based build.
- Use in-place construction instead of assignment.
- `FilteringSourceAccessor::checkAccess` had a fallback for when
`makeNotAllowedError` was null. Since `MakeNotAllowedError` is now
`fun<>`, we've proven the null branch is dead code and have removed it.
- `src/nix/flake.cc`, `src/nix/search.cc`, `src/nix/ls.cc` are left
as-is — the self-referential lambdas there cause too much
reindentation for too little benefit.
152d7a9b48