[lkml]   [2010]   [Jul]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    SubjectRe: Possible false positive from
    On Mon, 2010-07-12 at 13:35 -0500, Larry Finger wrote:
    > These are ugly macros that will be eliminated, but for the moment they are in
    > the code. As I stated in my original email, removing the comma from the
    > definition and adding it to the code does fix the checkpatch error, but it
    > should not be necessary.

    Hi Larry.

    Using checkpatch is not necessary.

    If you want generally conforming kernel style,
    the macro should not end in a trailing comma.

    Feel free to ignore the checkpatch message,

    I think the warning is reasonable, though it
    could be made more specific.

    cheers, Joe

    Maybe something like:
    diff --git a/scripts/ b/scripts/
    index bd88f11..7e8a3f4 100755
    --- a/scripts/
    +++ b/scripts/
    @@ -2394,8 +2394,10 @@ sub process {
    #print "REST<$rest> dstat<$dstat>\n";
    if ($rest ne '') {
    - if ($rest !~ /while\s*\(/ &&
    - $dstat !~ /$exceptions/)
    + if ($rest eq ",") {
    + ERROR("Macros should not end with a trailing comma\n" . "$here\n$ctx\n");
    + } elsif ($rest !~ /while\s*\(/ &&
    + $dstat !~ /$exceptions/)
    ERROR("Macros with multiple statements should be enclosed in a do - while loop\n" . "$here\n$ctx\n");

     \ /
      Last update: 2010-07-12 21:05    [W:0.022 / U:7.980 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site