On 02/16/2015 01:31 PM, Pádraig Brady wrote:
> On 15/02/15 22:26, Bernhard Voelker wrote:
>> diff --git a/src/du.c b/src/du.c
>> index 65fc074..5c664f9 100644
>> --- a/src/du.c
>> +++ b/src/du.c
>> @@ -968,8 +968,7 @@ main (int argc, char **argv)
>> {
>> /* Ignore "posix-" prefix, for compatibility with ls. */
>> static char const posix_prefix[] = "posix-";
>> - while (strncmp (time_style, posix_prefix, sizeof posix_prefix
>> - 1)
>> - == 0)
>> + while (STRPREFIX (time_style, posix_prefix))
>> time_style += sizeof posix_prefix - 1;
>
> I think the only difference between STREQ_LEN() and STRPREFIX()
> is that the former is more efficient when len is available?
> Hence STREQ_LEN() could be used above too?
Good point - pushed with that adjustment:
http://git.sv.gnu.org/cgit/coreutils.git/commit/?id=9f5dce8070
Thanks for the review.
Have a nice day,
Berny
