mail
mail copied to clipboard
get rid of rescue nil
not a good idea, nil check should be enough
Believe this is meant to catch parsing errors. Agree that blanket rescue nil
is too broad, though. We can be more precise.
I tested this change with 3 test suites (mail, actionmailer and redmine) wihout any regression. These methods aren't responsible for parsing, it should be just a lookup. Rescuing StandardError is too cautious, but well, still better than rescue Exception 😈
rebased