Hi Doug,
On Thu, Mar 21, 2013 at 9:50 AM, Doug Anderson dianders@chromium.org wrote:
Simon,
On Wed, Mar 20, 2013 at 7:42 PM, Simon Glass sjg@chromium.org wrote:
# Separates a tag: at the beginning of the subject from the rest of it -re_subject_tag = re.compile('([^:]*):\s*(.*)') +re_subject_tag = re.compile('([^:"]*):\s*(.*)')
I'd go further and prevent all spaces. re_subject_tag = re.compile('([^:\s]*):\s*(.*)')
I ran into a similar problem with my patch I sent up earlier:
patman: Add Cover-letter-cc: tag to Cc cover letter to people
I "fixed" that by removing the extra colon, but that was sorta lame. We shouldn't have tags with spaces in them anyway...
patman: Add Cover-letter-cc tag to Cc cover letter to people
Yes - will update this, thanks.
Regards, Simon