lkml.org 
[lkml]   [2007]   [Aug]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] update checkpatch.pl to version 0.08
    Jan Engelhardt wrote:
    > On Jul 23 2007 16:36, Kok, Auke wrote:
    >> this somehow seems to match something completely non-related (a function
    >> pointer declaration case):
    >>
    >> ERROR: no space between function name and open parenthesis '('
    >> #7278: FILE: drivers/net/e1000e/hw.h:434:
    >> + bool (*check_mng_mode)(struct e1000_hw *);
    >>
    >> even if I put a space between ")(", it still complains.
    >
    > [assumed] majority of kernel code uses ")(" instead of ")\s+(",
    > so the warning seems bogus. And it even gets nitpicky.

    No that is just a false positive. Those are meant to be detected, its
    just that bool wasn't listed as a type.

    -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/

    \
     
     \ /
      Last update: 2007-08-03 14:41    [W:0.020 / U:0.164 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site