diff options
| author | Dana Robinson <43805+derobins@users.noreply.github.com> | 2022-03-17 14:53:19 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-03-17 14:53:19 (GMT) |
| commit | 96398e47af98c55e14b81527a354c2f83c370576 (patch) | |
| tree | 6cec155318c0550286f7738b712164e163678e43 /bin | |
| parent | 34442f37400b0090cf56179f3851b21d7950f91c (diff) | |
| download | hdf5-96398e47af98c55e14b81527a354c2f83c370576.zip hdf5-96398e47af98c55e14b81527a354c2f83c370576.tar.gz hdf5-96398e47af98c55e14b81527a354c2f83c370576.tar.bz2 | |
Misc fixes highlighted by the checkposix script (#1499)
Diffstat (limited to 'bin')
| -rwxr-xr-x | bin/checkposix | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/bin/checkposix b/bin/checkposix index 36d07a8..bca259d 100755 --- a/bin/checkposix +++ b/bin/checkposix @@ -115,7 +115,7 @@ foreach $arg (@ARGV) { # These are really HDF5 functions/macros even though they don't # start with `h' or `H'. - next if $name =~ /^FUNC_(ENTER|LEAVE)(_(NO)?API|_PACKAGE|_STATIC)?(_NOFS|_NOCLEAR|_NOINIT|_NOPUSH)?(_NOFUNC|_TAG)?$/; + next if $name =~ /^FUNC_(ENTER|LEAVE)(_(NO)?API|_PACKAGE|_STATIC)?(_NAMECHECK_ONLY|_NOFS|_NOCLEAR|_NOINIT|_NOPUSH)?(_NOFUNC|_TAG)?$/; next if $name =~ /^(BEGIN|END)_FUNC$/; next if $name =~ /^U?INT(8|16|32|64)(ENCODE|DECODE)(_VAR)?$/; next if $name =~ /^CI_(PRINT_STATS|INC_SRC|INC_DST)$/; @@ -123,10 +123,13 @@ foreach $arg (@ARGV) { next if $name =~ /^(MIN3?|MAX3?|NELMTS|POWER_OF_TWO|REGION_OVERFLOW)$/; next if $name =~ /^(SIZE_OVERFLOW|UNIQUE_MEMBERS|S_ISDIR)$/; next if $name =~ /^addr_defined$/; + next if $name =~ /^TERMINATOR$/; - # These functions/macros are exempt. - # op, cb, and OP are often spuriously flagged so ignore them. - next if $name =~ /^(main|op|cb|OP)$/; + # Ignore callback invocation + next if $name =~ /^(op|cb|OP|iter_op|func)$/; + + # Ignore main + next if $name =~ /^main$/; # This often appears in preprocessor lines that span multiple lines next if $name =~ /^(defined)$/; |