On 2/12/2013 11:13 AM, Junio C Hamano wrote:
> Brandon Casey <draf...@gmail.com> writes:
>
>> When 'cherry-pick -s' is used to append a signed-off-by line to a cherry
>> picked commit, it does not currently detect the "(cherry picked from..."
>> that may have been appended by a previous 'cherry-pick -x' as part of the
>> s-o-b footer and it will insert a blank line before appending a new s-o-b.
>>
>> Let's detect "(cherry picked from...)" as part of the footer so that we
>> will produce this:
>> ...
>> +static int is_cherry_picked_from_line(const char *buf, int len)
>> +{
>> + /*
>> + * We only care that it looks roughly like (cherry picked from ...)
>> + */
>> + return len > strlen(cherry_picked_prefix) + 1 &&
>> + !prefixcmp(buf, cherry_picked_prefix) && buf[len - 1] == ')';
>> +}
>
> Does the first "is it longer than the prefix?" check matter? If it
> is not, prefixcmp() would not match anyway, no?
Probably not in practice, but technically we should only be accessing
len characters in buf even though buf may be longer than len. So the
check is just making sure the function doesn't access chars it's not
supposed to.
-Brandon
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may
contain
confidential information. Any unauthorized review, use, disclosure or
distribution
is prohibited. If you are not the intended recipient, please contact the
sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
