Re: [PATCH 4/5] pretty: Add name_and_address_only parameter

From: Calvin Wan <calvinwan@google.com>
Date: Feb 1, 2023, 23:52 UTC
Message-ID: <20230201235253.1891546-4-calvinwan@google.com>
In-Reply-To: <20230119223858.29262-5-zev@bewilderbeest.net>

Zev Weiss <zev@bewilderbeest.net> writes:

Show 5 quoted lines

> This is meant to be used with pp_user_info() when using it to format
> email recipients generated by --to-cmd/--cc-cmd.  When set it omits
> the leading 'From: ', trailing linefeed, and the date suffix, and
> additionally will return the input string unmodified if
> split_ident_line() can't parse it (e.g. for a bare email address).

I think the ideal solution, instead of choosing this hacky approach, is to refactor out the common functionality you need from pp_user_info() and to call that instead in your next patch.