Tests: Check for and remove trailing newlines

src/V3Global.h

@@ -125,5 +125,4 @@ extern V3Global v3Global;
 
 //######################################################################
 
-#endif // guard
+#endif  // guard
-

src/V3Hashed.cpp

@@ -217,4 +217,3 @@ V3Hashed::iterator V3Hashed::findDuplicate(AstNode* nodep, V3HashedUserCheck* ch
     }
     return end();
 }
-

src/V3LanguageWords.h

@@ -186,5 +186,4 @@ class V3LanguageWords {
     }
 };
 
-#endif // Guard
+#endif  // Guard
-

src/verilog.l

@@ -1127,4 +1127,3 @@ int V3ParseImp::lexToBison() {
     }
     return yylval.token;
 }
-

test_regress/input.vc

@@ -5,4 +5,3 @@
  +incdir+t
  +incdir+../include
  +incdir+obj_dir/
-

test_regress/t/t_bind2.v

@@ -89,4 +89,3 @@ module mycheck (/*AUTOARG*/
       $finish;
    end
 endmodule
-

test_regress/t/t_case_dupitems.v

@@ -78,4 +78,3 @@ module Test (/*AUTOARG*/
       endcase
    end
 endmodule
-

test_regress/t/t_case_reducer.v

@@ -276,4 +276,3 @@ module clz(
   assign out = {4'b0000, clz_byte1};
 
 endmodule // clz
-

test_regress/t/t_cellarray.v

@@ -97,4 +97,3 @@ module drv4
    assign datao = colSelB & colSelA & colSelC & colSelD;
 
 endmodule
-

test_regress/t/t_clk_inp_init.cpp

@@ -63,5 +63,3 @@ int main(int argc, char **argv, char **env) {
 
     return 0;
 }
-
-

test_regress/t/t_concat_opt.v

@@ -68,4 +68,3 @@ module t (/*AUTOARG*/
    end
 
 endmodule
-

test_regress/t/t_display_bad.pl

@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_dist_whitespace.pl

@@ -22,9 +22,14 @@ foreach my $file (sort keys %files) {
         # Ignore golden files
     } elsif ($contents =~ /[\001\002\003\004\005\006]/) {
         # Ignore binrary files
-    } elsif ($contents =~ /[ \t]\n/) {
+    } elsif ($contents =~ /[ \t]\n/
+             || $contents =~ m/\n\n+$/) {  # Regexp repeated below
+        my $eol_ws_exempt = ($file =~ /(\.txt|\.html)$/
+                             || $file =~ m!^README$!
+                             || $file =~ m!/gtkwave/!);
         if ($ENV{HARNESS_UPDATE_GOLDEN}) {
             $contents =~ s/[ \t]+\n/\n/g;
+            $contents =~ s/\n\n+$/\n/g unless $eol_ws_exempt;
             $warns{$file} = "Updated whitespace at $file";
             write_wholefile($filename, $contents);
             next;
@@ -41,6 +46,9 @@ foreach my $file (sort keys %files) {
                 $warns{$file} .= " (last character is ASCII " . ord(substr($line, -1, 1)) . ")";
             }
         }
+        if ($contents =~ m/\n\n+$/ && !$eol_ws_exempt) {  # Regexp repeated above
+            $warns{$file} = "Trailing newlines at EOF in $file";
+        }
     }
 }
 

test_regress/t/t_enum_public.v

@@ -30,4 +30,3 @@ module t (/*AUTOARG*/);
    end
 
 endmodule
-

test_regress/t/t_flag_debug_noleak.v

@@ -9,4 +9,3 @@ module t;
       $finish;
    end
 endmodule
-

test_regress/t/t_flag_f__2.vc

@@ -7,4 +7,3 @@
    */
 
 +define+GOT_DEF2=1
-

test_regress/t/t_flag_parameter.vc

@@ -12,4 +12,3 @@
 -pvalue+int32=123
 -Gint41=32'hdead_beef
 -pvalue+int42=32'hdead_beef
-

test_regress/t/t_flag_woff.v

@@ -17,4 +17,3 @@ module t;
    wire C = 15'd1234;
 
 endmodule
-

test_regress/t/t_func_bad.pl

@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_func_bad2.pl

@@ -16,4 +16,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_func_default_warn.v

@@ -25,4 +25,3 @@ module t (/*AUTOARG*/);
    end
 
 endmodule
-

test_regress/t/t_func_defaults.v

@@ -30,4 +30,3 @@ module t (/*AUTOARG*/);
    end
 
 endmodule
-

test_regress/t/t_gate_implicit.v

@@ -77,4 +77,3 @@ endmodule
 module Mxor (output out, input a, b);
    assign out = (a ^ b);
 endmodule
-

test_regress/t/t_gen_div0.v

@@ -36,4 +36,3 @@ module Test ();
         end
    endgenerate
 endmodule
-

test_regress/t/t_genvar_misuse_bad.v

@@ -8,10 +8,9 @@ module top
   (
    output logic [1:0] q,
    input logic [1:0]  d,
-   input logic 	      clk
+   input logic        clk
    );
 
-   genvar 	     i;
+   genvar             i;
-   assign 	     q[i] = d[i];
+   assign            q[i] = d[i];
 endmodule
-

test_regress/t/t_interface_missing_bad.v

@@ -30,4 +30,3 @@ module t (/*AUTOARG*/);
    end
 
 endmodule
-

test_regress/t/t_lint_declfilename.pl

@@ -18,4 +18,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_declfilename_bad.pl

@@ -25,4 +25,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_defparam.pl

@@ -18,4 +18,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_defparam_bad.pl

@@ -25,4 +25,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_ifdepth_bad.pl

@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_implicit.pl

@@ -15,4 +15,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_implicit_bad.pl

@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_implicit_def_bad.pl

@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_implicit_port.pl

@@ -15,4 +15,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_importstar_bad.pl

@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_incabspath.pl

@@ -18,4 +18,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_incabspath_bad.pl

@@ -25,4 +25,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_latch_bad.pl

@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_subout_bad.pl

@@ -18,4 +18,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_syncasyncnet_bad.pl

@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_unused.pl

@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_unused_bad.pl

@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_unused_iface.pl

@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_lint_unused_iface_bad.pl

@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_math_sign_extend.v

@@ -129,4 +129,3 @@ module t (/*AUTOARG*/
 
    end
 endmodule
-

test_regress/t/t_param_func.v

@@ -31,4 +31,3 @@ module test#(
     endfunction
 
 endmodule
-

test_regress/t/t_param_sel_range.v

@@ -46,5 +46,3 @@ module submod (/*AUTOARG*/
    wire out = r_rst[STAGES-1];
 
 endmodule
-
-

test_regress/t/t_past_bad.pl

@@ -16,4 +16,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_past_unsup_bad.pl

@@ -18,4 +18,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_pp_circdef_bad.pl

@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_pp_dupdef.pl

@@ -15,4 +15,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_pp_dupdef_bad.pl

@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_pp_lib.pl

@@ -19,4 +19,3 @@ execute(
 
 ok(1);
 1;
-

test_regress/t/t_pp_misdef_bad.pl

@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_pp_underline_bad.pl

@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_preproc_persist_inc.v

@@ -7,4 +7,3 @@
  `define COMMON_GUARD 1
 Inside `__FILE__.
 `endif
-

test_regress/t/t_select_bad_msb.pl

@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_select_bad_range.pl

@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_select_bad_range2.pl

@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_select_bad_range3.pl

@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_select_bad_tri.pl

@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_select_index.pl

@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-

test_regress/t/t_select_param.pl

@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-

test_regress/t/t_select_runtime_range.pl

@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-

test_regress/t/t_select_set.pl

@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-

test_regress/t/t_struct_anon.v

@@ -23,4 +23,3 @@ module t (
    initial struct2 = 64'h123456789_abcdef0;
    always_comb s2 = struct2;
 endmodule
-

test_regress/t/t_struct_init.pl

@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-

test_regress/t/t_struct_init_bad.pl

@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_struct_nest.v

@@ -42,4 +42,3 @@ module t (
    always_comb datao_pre = { xxxxxxx_yyyyy_zzzz.dddddd.b1.fbyte, xxxxxxx_yyyyy_zzzz.dddddd.c1 };
 
 endmodule
-

test_regress/t/t_struct_pat_width.pl

@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-

test_regress/t/t_struct_port.pl

@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-

test_regress/t/t_sys_plusargs_bad.pl

@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-

test_regress/t/t_sys_readmem_bad_digit.mem

@@ -6,4 +6,3 @@
 // Version 2.0.
 
 a0
-

test_regress/t/t_trace_fst.pl

@@ -19,4 +19,3 @@ vcd_identical("$Self->{obj_dir}/simx-fst2vcd.vcd", $Self->{golden_filename});
 
 ok(1);
 1;
-

test_regress/t/t_tri_ifbegin.v

@@ -49,4 +49,3 @@ module sub (/*AUTOARG*/
    endgenerate
 
 endmodule
-

test_regress/t/t_tri_inout2.v

@@ -73,4 +73,3 @@ endmodule
 module ChildB(input A, output Y);
   assign Y = A;
 endmodule
-

test_regress/t/t_tri_pull2_bad.pl

@@ -16,4 +16,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_tri_pull_bad.pl

@@ -16,4 +16,3 @@ compile(
 
 ok(1);
 1;
-

test_regress/t/t_tri_various.v

@@ -208,4 +208,3 @@ endmodule
 //      end
 //   end
 //endmodule
-

test_regress/t/t_typedef_array.v

@@ -23,4 +23,3 @@ module t;
       $finish;
    end
 endmodule
-