Re: [PATCH 2/4] xfsprogs: Addition in debug option -d of xfs_spaceman

[Dhruvesh Rathore <adrscube@xxxxxxxxx>]

On Wed, Feb 11, 2015 at 3:44 AM, Dave Chinner <david@xxxxxxxxxxxxx> wrote:
> On Thu, Jan 29, 2015 at 06:55:21PM +0530, Dhruvesh Rathore wrote:
>>
>> The xfs_spaceman utility did not have a a proper title to the content
>> displayed using -d (debug) option.
> .....
>>
>> @@ -332,20 +332,23 @@
>>  static int
>>  freesp_f(
>>       int             argc,
>>       char            **argv)
>>  {
>>       xfs_agnumber_t  agno;
>>
>>       if (!init(argc, argv))
>>               return 0;
>>
>> +     if (dumpflag)
>> +             printf("%8s %8s %8s\n", "agno", "agbno", "len");
>> +
>>       for (agno = 0; agno < file->geom.agcount; agno++)  {
>>               if (inaglist(agno))
>>                       scan_ag(agno);
>>       }
>>       if (histcount)
>>               printhist();
>
> Patch does not apply to the current tree. It's got stray whitespace
> and lines in it that are in the original source file.
>
> I've fixed this up for the moment, but please make sure your
> patches apply to a clean tree.
>

We apologize for the constant stray white space errors and will
make sure to make clean patches from now on.
Thank you for reworking and applying the patch to the current tree :)

Regards,
A-DRS