checkpatch.pl print the line twice when trailing statements in the same line - Kernel

This is a discussion on checkpatch.pl print the line twice when trailing statements in the same line - Kernel ; Hi. I executed checkpatch.pl version 0.24 on a file and i noticed that when a tailing statement is being declared at the same line with "while, checkpatch prints the erroneous line twice. Here is an example of the output: ERROR: ...

+ Reply to Thread
Results 1 to 2 of 2

Thread: checkpatch.pl print the line twice when trailing statements in the same line

  1. checkpatch.pl print the line twice when trailing statements in the same line

    Hi.

    I executed checkpatch.pl version 0.24 on a file and i noticed that when
    a tailing statement is being declared at the same line with "while, checkpatch
    prints the erroneous line twice.

    Here is an example of the output:

    ERROR: trailing statements should be on next line
    #651: FILE: idbm.c:651:
    + while (*nl && isspace(c = *nl)) nl++;
    + while (*nl && isspace(c = *nl)) nl++;

    Is this is the expected behavior?

    Thanks
    Dotan
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  2. Re: checkpatch.pl print the line twice when trailing statements in the same line

    On Mon, Oct 20, 2008 at 08:51:38PM +0200, Dotan Barak wrote:
    > Hi.
    >
    > I executed checkpatch.pl version 0.24 on a file and i noticed that when
    > a tailing statement is being declared at the same line with "while, checkpatch
    > prints the erroneous line twice.
    >
    > Here is an example of the output:
    >
    > ERROR: trailing statements should be on next line
    > #651: FILE: idbm.c:651:
    > + while (*nl && isspace(c = *nl)) nl++;
    > + while (*nl && isspace(c = *nl)) nl++;
    >
    > Is this is the expected behavior?


    No that would definintaly not be expected. Will add it to my TODO list.

    Thanks for the report.

    -apw
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

+ Reply to Thread