fbrosson | 533407a | 2018-04-04 21:44:29 +0000 | [diff] [blame] | 1 | #!/usr/bin/env perl |
Paul Bakker | 9a73632 | 2012-11-14 12:39:52 +0000 | [diff] [blame] | 2 | |
| 3 | # Detect comment blocks that are likely meant to be doxygen blocks but aren't. |
| 4 | # |
| 5 | # More precisely, look for normal comment block containing '\'. |
| 6 | # Of course one could use doxygen warnings, eg with: |
Manuel Pégourié-Gonnard | f234ff8 | 2015-01-22 17:01:27 +0000 | [diff] [blame] | 7 | # sed -e '/EXTRACT/s/YES/NO/' doxygen/mbedtls.doxyfile | doxygen - |
Paul Bakker | 9a73632 | 2012-11-14 12:39:52 +0000 | [diff] [blame] | 8 | # but that would warn about any undocumented item, while our goal is to find |
| 9 | # items that are documented, but not marked as such by mistake. |
Bence Szépkúti | 700ee44 | 2020-05-26 00:33:31 +0200 | [diff] [blame] | 10 | # |
Bence Szépkúti | 1e14827 | 2020-08-07 13:07:28 +0200 | [diff] [blame] | 11 | # Copyright The Mbed TLS Contributors |
Dave Rodgman | 16799db | 2023-11-02 19:47:20 +0000 | [diff] [blame] | 12 | # SPDX-License-Identifier: Apache-2.0 OR GPL-2.0-or-later |
Paul Bakker | 9a73632 | 2012-11-14 12:39:52 +0000 | [diff] [blame] | 13 | |
| 14 | use warnings; |
| 15 | use strict; |
| 16 | use File::Basename; |
| 17 | |
Manuel Pégourié-Gonnard | d09a6b5 | 2015-04-09 17:19:23 +0200 | [diff] [blame] | 18 | # C/header files in the following directories will be checked |
Manuel Pégourié-Gonnard | 7f80997 | 2015-03-09 17:05:11 +0000 | [diff] [blame] | 19 | my @directories = qw(include/mbedtls library doxygen/input); |
Paul Bakker | 9a73632 | 2012-11-14 12:39:52 +0000 | [diff] [blame] | 20 | |
| 21 | # very naive pattern to find directives: |
Manuel Pégourié-Gonnard | ef009ff | 2013-09-16 13:40:25 +0200 | [diff] [blame] | 22 | # everything with a backslach except '\0' and backslash at EOL |
| 23 | my $doxy_re = qr/\\(?!0|\n)/; |
Paul Bakker | 9a73632 | 2012-11-14 12:39:52 +0000 | [diff] [blame] | 24 | |
Andres Amaya Garcia | d3f0f5e | 2016-12-14 09:36:55 +0000 | [diff] [blame] | 25 | # Return an error code to the environment if a potential error in the |
| 26 | # source code is found. |
| 27 | my $exit_code = 0; |
| 28 | |
Paul Bakker | 9a73632 | 2012-11-14 12:39:52 +0000 | [diff] [blame] | 29 | sub check_file { |
| 30 | my ($fname) = @_; |
| 31 | open my $fh, '<', $fname or die "Failed to open '$fname': $!\n"; |
| 32 | |
| 33 | # first line of the last normal comment block, |
| 34 | # or 0 if not in a normal comment block |
| 35 | my $block_start = 0; |
| 36 | while (my $line = <$fh>) { |
| 37 | $block_start = $. if $line =~ m/\/\*(?![*!])/; |
| 38 | $block_start = 0 if $line =~ m/\*\//; |
| 39 | if ($block_start and $line =~ m/$doxy_re/) { |
| 40 | print "$fname:$block_start: directive on line $.\n"; |
| 41 | $block_start = 0; # report only one directive per block |
Andres Amaya Garcia | d3f0f5e | 2016-12-14 09:36:55 +0000 | [diff] [blame] | 42 | $exit_code = 1; |
Paul Bakker | 9a73632 | 2012-11-14 12:39:52 +0000 | [diff] [blame] | 43 | } |
| 44 | } |
| 45 | |
| 46 | close $fh; |
| 47 | } |
| 48 | |
| 49 | sub check_dir { |
| 50 | my ($dirname) = @_; |
| 51 | for my $file (<$dirname/*.[ch]>) { |
| 52 | check_file($file); |
| 53 | } |
| 54 | } |
| 55 | |
Andres Amaya Garcia | d3f0f5e | 2016-12-14 09:36:55 +0000 | [diff] [blame] | 56 | # Check that the script is being run from the project's root directory. |
Paul Bakker | 9a73632 | 2012-11-14 12:39:52 +0000 | [diff] [blame] | 57 | for my $dir (@directories) { |
Andres Amaya Garcia | d3f0f5e | 2016-12-14 09:36:55 +0000 | [diff] [blame] | 58 | if (! -d $dir) { |
Gilles Peskine | e820c0a | 2023-08-03 17:45:20 +0200 | [diff] [blame] | 59 | die "This script must be run from the Mbed TLS root directory"; |
Andres Amaya Garcia | d3f0f5e | 2016-12-14 09:36:55 +0000 | [diff] [blame] | 60 | } else { |
| 61 | check_dir($dir) |
| 62 | } |
Paul Bakker | 9a73632 | 2012-11-14 12:39:52 +0000 | [diff] [blame] | 63 | } |
| 64 | |
Andres Amaya Garcia | d3f0f5e | 2016-12-14 09:36:55 +0000 | [diff] [blame] | 65 | exit $exit_code; |
| 66 | |
Paul Bakker | 9a73632 | 2012-11-14 12:39:52 +0000 | [diff] [blame] | 67 | __END__ |