Если у вас (или у вашего тимлида) нет
ОКР, то можно и suppress сделать.
НО!
Тут при повторном рассмотрении углядел, что вы в obj передаёте не ссылку на объект, который событие отправляет, а какой-то параметр. В этом случае это действительно может ввести в заблуждение (как ввело меня только что, например, когда я подумал, что вы параметры не передаёте, а только ссылку на отправителя события). Я, не зная вашего кода, вижу событие и ОЖИДАЮ стандартного шаблона (особенно если вы используете тип object), и здесь мой шаблон "рвётся" :)
Во-первых, есть класс
ProgressEventArgs, который служит ровно той цели, что вы здесь "свелосипедили".
Стиль написания кода - это, если уж честно, догма от Майкрософт, и вещь субъективная. Однако, нельзя считать эту догму плохой. Она задаёт общий шаблон для написания кода и позволяет потом легко свои решения масштабировать.
В общем, это ваше решение. Если код закрытый и никто, кроме вас его больше не увидит, и вы уверены, что и через несколько лет вы, просматривая этот фрагмент, будете чётко знать, что на самом деле в объекте obj типа object вы передаёте в событии целочисленный параметр состояния прогресса, то можете делать suppress этому кейсу.
Если же вы планируете публиковать этот код (это часть Open Source) проекта, то вердикт однозначный - переписывайте.