[SUGGESTION] Checking the dialogResult logic


I had a bug caused by an inattention.

if (MessageBox.Show(LocRM.GetString("sendCancelEmail"), "", MessageBoxButtons.YesNo) == DialogResult.OK)

In this case DialogResult.OK is an impossible value, so the condition would never be true. I think ReSharper should check this one. So maybe for the next release / update.

What do you think?

Thank you!

Please sign in to leave a comment.