Why do we need to trim this?  Whatever is between < and > should be taken as-is as the email.  Do you have an example of a malformed commit where the email contains leading or trailing whitespace near the <>'s and it needs to be discarded?  Even if that exists, I'd almost still be tempted to say that is the proper text of the address, and we should preserve that text in the parsing.