blob: b0fd69635521c45e4b6a990d14bb464aef6b4c40 [file] [log] [blame]
Paul Bakker9a736322012-11-14 12:39:52 +00001#!/usr/bin/perl
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é-Gonnardf234ff82015-01-22 17:01:27 +00007# sed -e '/EXTRACT/s/YES/NO/' doxygen/mbedtls.doxyfile | doxygen -
Paul Bakker9a736322012-11-14 12:39:52 +00008# 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.
10
11use warnings;
12use strict;
13use File::Basename;
14
Manuel Pégourié-Gonnardd09a6b52015-04-09 17:19:23 +020015# C/header files in the following directories will be checked
Manuel Pégourié-Gonnard7f809972015-03-09 17:05:11 +000016my @directories = qw(include/mbedtls library doxygen/input);
Paul Bakker9a736322012-11-14 12:39:52 +000017
18# very naive pattern to find directives:
Manuel Pégourié-Gonnardef009ff2013-09-16 13:40:25 +020019# everything with a backslach except '\0' and backslash at EOL
20my $doxy_re = qr/\\(?!0|\n)/;
Paul Bakker9a736322012-11-14 12:39:52 +000021
Andres Amaya Garciad3f0f5e2016-12-14 09:36:55 +000022# Return an error code to the environment if a potential error in the
23# source code is found.
24my $exit_code = 0;
25
Paul Bakker9a736322012-11-14 12:39:52 +000026sub check_file {
27 my ($fname) = @_;
28 open my $fh, '<', $fname or die "Failed to open '$fname': $!\n";
29
30 # first line of the last normal comment block,
31 # or 0 if not in a normal comment block
32 my $block_start = 0;
33 while (my $line = <$fh>) {
34 $block_start = $. if $line =~ m/\/\*(?![*!])/;
35 $block_start = 0 if $line =~ m/\*\//;
36 if ($block_start and $line =~ m/$doxy_re/) {
37 print "$fname:$block_start: directive on line $.\n";
38 $block_start = 0; # report only one directive per block
Andres Amaya Garciad3f0f5e2016-12-14 09:36:55 +000039 $exit_code = 1;
Paul Bakker9a736322012-11-14 12:39:52 +000040 }
41 }
42
43 close $fh;
44}
45
46sub check_dir {
47 my ($dirname) = @_;
48 for my $file (<$dirname/*.[ch]>) {
49 check_file($file);
50 }
51}
52
Andres Amaya Garciad3f0f5e2016-12-14 09:36:55 +000053# Check that the script is being run from the project's root directory.
Paul Bakker9a736322012-11-14 12:39:52 +000054for my $dir (@directories) {
Andres Amaya Garciad3f0f5e2016-12-14 09:36:55 +000055 if (! -d $dir) {
56 die "This script must be run from the mbed TLS root directory";
57 } else {
58 check_dir($dir)
59 }
Paul Bakker9a736322012-11-14 12:39:52 +000060}
61
Andres Amaya Garciad3f0f5e2016-12-14 09:36:55 +000062exit $exit_code;
63
Paul Bakker9a736322012-11-14 12:39:52 +000064__END__