From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 25 Nov 2009 01:10:57 GMT
Subject: Re: [PATCH] mailinfo: remove [PATCH...] prefix from Subject regardless of length
Message-ID: <7vmy2b76ji.fsf@alter.siamese.dyndns.org>
In-Reply-To: <874ookp4u8.fsf@meyering.net>

Jim Meyering <meyering@redhat.com> writes:

> Before this change, a [...] prefix would be removed only as long as
> its length did not exceed 2/3 of the subject length.  Now, when the
> bracketed quantity starts with PATCH, it is removed unconditionally.
> Otherwise, the existing behavior remains unchanged.

Thanks, I think this is a good idea in general, but have two comments.

 - I am not sure how this should play with 17635fc (mailinfo: -b option
   keeps [bracketed] strings that is not a [PATCH] marker, 2009-07-15).

 - Regardless of interaction with 17635fc, Things like [RFC PATCH]
   [SECURITY PATCH] might want a similar treatment.