blob: 3ed7069c57e088706c223ebd6493989aedf09b65 [file] [log] [blame]
fbrosson533407a2018-04-04 21:44:29 +00001#!/usr/bin/env perl
Paul Bakker9a736322012-11-14 12:39:52 +00002
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.
Bence Szépkúti700ee442020-05-26 00:33:31 +020010#
Bence Szépkúti1e148272020-08-07 13:07:28 +020011# Copyright The Mbed TLS Contributors
Bence Szépkútic7da1fe2020-05-26 01:54:15 +020012# SPDX-License-Identifier: Apache-2.0
13#
14# Licensed under the Apache License, Version 2.0 (the "License"); you may
15# not use this file except in compliance with the License.
16# You may obtain a copy of the License at
17#
18# http://www.apache.org/licenses/LICENSE-2.0
19#
20# Unless required by applicable law or agreed to in writing, software
21# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
22# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
23# See the License for the specific language governing permissions and
24# limitations under the License.
Paul Bakker9a736322012-11-14 12:39:52 +000025
26use warnings;
27use strict;
28use File::Basename;
29
Manuel Pégourié-Gonnardd09a6b52015-04-09 17:19:23 +020030# C/header files in the following directories will be checked
Manuel Pégourié-Gonnard7f809972015-03-09 17:05:11 +000031my @directories = qw(include/mbedtls library doxygen/input);
Paul Bakker9a736322012-11-14 12:39:52 +000032
33# very naive pattern to find directives:
Manuel Pégourié-Gonnardef009ff2013-09-16 13:40:25 +020034# everything with a backslach except '\0' and backslash at EOL
35my $doxy_re = qr/\\(?!0|\n)/;
Paul Bakker9a736322012-11-14 12:39:52 +000036
Andres Amaya Garciad3f0f5e2016-12-14 09:36:55 +000037# Return an error code to the environment if a potential error in the
38# source code is found.
39my $exit_code = 0;
40
Paul Bakker9a736322012-11-14 12:39:52 +000041sub check_file {
42 my ($fname) = @_;
43 open my $fh, '<', $fname or die "Failed to open '$fname': $!\n";
44
45 # first line of the last normal comment block,
46 # or 0 if not in a normal comment block
47 my $block_start = 0;
48 while (my $line = <$fh>) {
49 $block_start = $. if $line =~ m/\/\*(?![*!])/;
50 $block_start = 0 if $line =~ m/\*\//;
51 if ($block_start and $line =~ m/$doxy_re/) {
52 print "$fname:$block_start: directive on line $.\n";
53 $block_start = 0; # report only one directive per block
Andres Amaya Garciad3f0f5e2016-12-14 09:36:55 +000054 $exit_code = 1;
Paul Bakker9a736322012-11-14 12:39:52 +000055 }
56 }
57
58 close $fh;
59}
60
61sub check_dir {
62 my ($dirname) = @_;
63 for my $file (<$dirname/*.[ch]>) {
64 check_file($file);
65 }
66}
67
Andres Amaya Garciad3f0f5e2016-12-14 09:36:55 +000068# Check that the script is being run from the project's root directory.
Paul Bakker9a736322012-11-14 12:39:52 +000069for my $dir (@directories) {
Andres Amaya Garciad3f0f5e2016-12-14 09:36:55 +000070 if (! -d $dir) {
71 die "This script must be run from the mbed TLS root directory";
72 } else {
73 check_dir($dir)
74 }
Paul Bakker9a736322012-11-14 12:39:52 +000075}
76
Andres Amaya Garciad3f0f5e2016-12-14 09:36:55 +000077exit $exit_code;
78
Paul Bakker9a736322012-11-14 12:39:52 +000079__END__