remove redundant if-else in needDiag, remove vars name1 & name2