Re: [PATCH] scripts: reduce false positives in the macro_checker script.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jan Kara <jack@xxxxxxx> 于2024年7月25日周四 04:52写道:
>
> On Thu 25-07-24 03:58:30, Julian Sun wrote:
> > Reduce false positives in the macro_checker
> > in the following scenarios:
> >   1. Conditional compilation
> >   2. Macro definitions with only a single character
> >   3. Macro definitions as (0) and (1)
> >
> > Before this patch:
> >       sjc@sjc:linux$ ./scripts/macro_checker.py  fs | wc -l
> >       99
> >
> > After this patch:
> >       sjc@sjc:linux$ ./scripts/macro_checker.py  fs | wc -l
> >       11
> >
> > Most of the current warnings are valid now.
> >
> > Signed-off-by: Julian Sun <sunjunchao2870@xxxxxxxxx>
> ...
> >  def file_check_macro(file_path, report):
> > +    # number of conditional compiling
> > +    cond_compile = 0
> >      # only check .c and .h file
> >      if not file_path.endswith(".c") and not file_path.endswith(".h"):
> >          return
> > @@ -57,7 +72,14 @@ def file_check_macro(file_path, report):
> >          while True:
> >              line = f.readline()
> >              if not line:
> > -                return
> > +                break
> > +            line = line.strip()
> > +            if line.startswith(cond_compile_mark):
> > +                cond_compile += 1
> > +                continue
> > +            if line.startswith(cond_compile_end):
> > +                cond_compile -= 1
> > +                continue
> >
> >              macro = re.match(macro_pattern, line)
> >              if macro:
> > @@ -67,6 +89,11 @@ def file_check_macro(file_path, report):
> >                      macro = macro.strip()
> >                      macro += f.readline()
> >                      macro = macro_strip(macro)
> > +                if file_path.endswith(".c")  and cond_compile != 0:
> > +                    continue
> > +                # 1 is for #ifdef xxx at the beginning of the header file
> > +                if file_path.endswith(".h") and cond_compile != 1:
> > +                    continue
> >                  check_macro(macro, report)
> >
> >  def get_correct_macros(path):
>
>
> > So I don't think this is right. As far as I understand this skips any macros
> > that are conditionally defined? Why? There is a lot of them and checking
> > them is beneficial... The patterns you have added should be dealing with
> > most of the conditional defines anyway.
Yes, this skips all checks for conditional macro. This is because I
observed that almost all false positives come from conditional
compilation. Testing showed that skipping them does not cause the
genuine warnings to disappear.
Also as you said, it may still lead to skipping checks for genuinely
problematic macro definitions. Perhaps we could provide an option that
allows users to control whether or not to check macros under
conditional compilation?
>
>                                                                 Honza
> --
> Jan Kara <jack@xxxxxxxx>
> SUSE Labs, CR


Thanks,
-- 
Julian Sun <sunjunchao2870@xxxxxxxxx>





[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux