Re: [PATCH 04/14] shortlog: use skip_prefix_icase to parse "Author" lines

On Thu, Dec 31, 2015 at 01:47:21AM -0500, Eric Sunshine wrote: > On Tue, Dec 29, 2015 at 2:27 AM, Jeff King wrote: > > Because we must match both "Author" and "author" here, we > > could not use skip_prefix, and had to hand-code a partial > > case-insensitive match. Now that we have skip_prefix_c

Re: [PATCH 04/14] shortlog: use skip_prefix_icase to parse "Author" lines

On Tue, Dec 29, 2015 at 2:27 AM, Jeff King wrote: > Because we must match both "Author" and "author" here, we > could not use skip_prefix, and had to hand-code a partial > case-insensitive match. Now that we have skip_prefix_case, s/skip_prefix_case/skip_prefix_icase/ > we can use it. This is te

[PATCH 04/14] shortlog: use skip_prefix_icase to parse "Author" lines

Because we must match both "Author" and "author" here, we could not use skip_prefix, and had to hand-code a partial case-insensitive match. Now that we have skip_prefix_case, we can use it. This is technically more liberal in what it matches (e.g., it will match AUTHOR), but in this particular case