GNU bug report logs - #27368
Minor concern: Confusing tail warning

Previous Next

Package: coreutils;

Reported by: Charlie Hagedorn <charlie.hagedorn <at> gmail.com>

Date: Wed, 14 Jun 2017 23:04:02 UTC

Severity: normal

Done: Pádraig Brady <P <at> draigBrady.com>

Bug is archived. No further changes may be made.

Full log


Message #28 received at 27368 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Pádraig Brady <P <at> draigbrady.com>
Cc: 27368 <at> debbugs.gnu.org
Subject: Re: bug#27368: Minor concern: Confusing tail warning
Date: Sat, 17 Jun 2017 16:47:24 -0700
On Sat, Jun 17, 2017 at 3:57 PM, Pádraig Brady <P <at> draigbrady.com> wrote:
> On 17/06/17 14:30, Pádraig Brady wrote:
>> On 17/06/17 07:35, Jim Meyering wrote:
>>> In this new function, please move the declaration of "i" into the for-loop:
>>>
>>> +static bool
>>> +any_non_regular (const struct File_spec *f, size_t n_files)
>>> +{
>>> +  size_t i;
>>> +
>>> +  for (i = 0; i < n_files; i++)
>
> I did that in about 100 instances at:
> http://git.sv.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=1379bdc

Nice! Thank you.




This bug report was last modified 8 years and 53 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.