Аллокации в циклах, sync-over-async, N+1, boxing, захваты лямбд, неограниченные кэши, отсутствие cancellation.
Разбор
- Спорные места — только с цифрами.
- Потокобезопасность границ.
- Регрессионные тесты.
- Читаемость без доказанной выгоды не жертвуют.
Итог
Perf-review = гипотезы на код + требование измерений.