You are preaching to the choir. I agree that MessageBox() should not throw the return code as @Error. As Jooel said, "setting it when there is no error is just plain wrong".