-
Notifications
You must be signed in to change notification settings - Fork 138
/
linelength.t
117 lines (90 loc) · 2.75 KB
/
linelength.t
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
#! perl
# Copyright (C) 2006-2009, Parrot Foundation.
=head1 NAME
t/codingstd/linelength.t - Test code lines length
=head1 SYNOPSIS
# test all files
% prove t/codingstd/linelength.t
# test specific files
% perl t/codingstd/linelength.t src/foo.c include/parrot/bar.h
=head1 DESCRIPTION
Tests source files for the line length limit as defined in
F<pdd07_codingstd.pod>. Only files for some language implementations are
checked.
=head1 SEE ALSO
L<docs/pdds/pdd07_codingstd.pod>
=cut
use strict;
use warnings;
use lib qw( . lib ../lib ../../lib );
use File::Spec;
use Test::More tests => 1;
use ExtUtils::Manifest qw( maniread );
use Parrot::Config qw/ %PConfig /;
my $num_col_limit = 100;
my $build_dir = $PConfig{build_dir};
my $manifest = maniread( File::Spec->catfile( $build_dir, 'MANIFEST' ) );
# skip files listed in the __DATA__ section
my %skip_files;
while (<DATA>) {
next if m{^#};
next if m{^\s*$};
chomp;
$skip_files{$_}++;
}
# find the files that we need to check
my @files = @ARGV ? <@ARGV> : source_files();
# check all the files and keep a list of those failing
my @lines_too_long;
foreach (@files) {
my $lineinfo = info_for_first_long_line($_);
next unless $lineinfo;
push @lines_too_long => $lineinfo;
}
## L<PDD07/Code Formatting/"Source line length is limited to 100 characters">
ok( !@lines_too_long, 'Line length ok' )
or diag( "Lines longer than coding standard limit ($num_col_limit columns) in "
. scalar @lines_too_long
. " files:\n"
. join( "\n", @lines_too_long ) );
sub info_for_first_long_line {
my $file = shift;
open my $fh, '<', $file or die "Can't open file '$file'";
while ( my $line = <$fh> ) {
chomp $line;
$line =~ s/\t/' ' x (1 + length($`) % 8)/eg; # expand \t
next if $line =~ m/https?:\/\//; # skip long web addresses
next if $line =~ m/CONST_STRING\(/;
return sprintf '%s:%d: %d cols', $file, $., length($line)
if length($line) > $num_col_limit;
}
return;
}
sub source_files {
my @files;
foreach my $file ( sort keys(%$manifest) ) {
my $full_path = File::Spec->catfile( $build_dir, $file );
# skip files specified in __DATA__ section
next if exists $skip_files{$file};
push @files => $full_path
if $file =~ m{\.(c|h|pmc|ops|pod)$};
}
return @files;
}
# Local Variables:
# mode: cperl
# cperl-indent-level: 4
# fill-column: 100
# End:
# vim: expandtab shiftwidth=4:
__DATA__
# Lex and Bison generated
compilers/imcc/imclexer.c
compilers/imcc/imcparser.c
# generated files
src/ops/core_ops.c
# generated by tools/dev/nci_thunk_gen.pir
src/nci/core_thunks.c
src/nci/extra_thunks.c
# these ones include long POD
docs/embed.pod