> Ну почему же, для черновой чистки кода, особенно после рефакторинга, когда классы обменивались членами и методами, вполне годится. Тут переменная уже не используется, тут - можно уменьшить ее область видимости, свичи без дефолтов лишний раз посмотреть - может, и нужно сделать... В общем, для "полировки" и в качестве имитации свежего взгляда - нормально. Как альтернатива ключам компилятора, в результате которых имеешь ворох замечаний, из которых 90% - вообще не к твоему коду, а к библиотечному.

Неиспользуемые переменные прекрасно отсекает gcc. И в этом он совершенно прав. Если, допустим, неиспользуемые аргументы метода вполне имеют право на жизнь (соответственно -Wno-unused-parameter валиден и правомерен) то неиспользуемые переменные - это откровенный мусор. Естественно, если это простой тип.

Свичи без дефолтов - тоже вполне валидны. При условии, что это перечисление и в свиче by design полный набор элементов. Это позволяет ещё на уровне компиляции обнаружить, что оказывается, перечисление внезапно! кто-то расширил новыми элементами а вот в селекторы тут там и сям их не добавил (забыл). Если бы стоял дефолт то компилятор бы это проглотил на ура. С самым разными последствиями.

Ворох же замечаний - он в подавляющем большинстве случаев не просто так. И не суть важно откуда они идут - из кода A или из стороннего кода B. Какая разница где именно грязь === риски багов? Если оно есть - с этим нужно разбираться, без вариантов.