From 7777d10e9baf43e602cd61cde5f8107137044106 Mon Sep 17 00:00:00 2001
From: Wilson Snyder <wsnyder@wsnyder.org>
Date: Mon, 13 May 2019 19:47:52 -0400
Subject: [PATCH] Tests: Check for and remove trailing newlines

---
 src/V3Global.h                             |  3 +--
 src/V3Hashed.cpp                           |  1 -
 src/V3LanguageWords.h                      |  3 +--
 src/verilog.l                              |  1 -
 test_regress/input.vc                      |  1 -
 test_regress/t/t_bind2.v                   |  1 -
 test_regress/t/t_case_dupitems.v           |  1 -
 test_regress/t/t_case_reducer.v            |  1 -
 test_regress/t/t_cellarray.v               |  1 -
 test_regress/t/t_clk_inp_init.cpp          |  2 --
 test_regress/t/t_concat_opt.v              |  1 -
 test_regress/t/t_display_bad.pl            |  1 -
 test_regress/t/t_dist_whitespace.pl        | 10 +++++++++-
 test_regress/t/t_enum_public.v             |  1 -
 test_regress/t/t_flag_debug_noleak.v       |  1 -
 test_regress/t/t_flag_f__2.vc              |  1 -
 test_regress/t/t_flag_parameter.vc         |  1 -
 test_regress/t/t_flag_woff.v               |  1 -
 test_regress/t/t_func_bad.pl               |  1 -
 test_regress/t/t_func_bad2.pl              |  1 -
 test_regress/t/t_func_default_warn.v       |  1 -
 test_regress/t/t_func_defaults.v           |  1 -
 test_regress/t/t_gate_implicit.v           |  1 -
 test_regress/t/t_gen_div0.v                |  1 -
 test_regress/t/t_genvar_misuse_bad.v       |  7 +++----
 test_regress/t/t_interface_missing_bad.v   |  1 -
 test_regress/t/t_lint_declfilename.pl      |  1 -
 test_regress/t/t_lint_declfilename_bad.pl  |  1 -
 test_regress/t/t_lint_defparam.pl          |  1 -
 test_regress/t/t_lint_defparam_bad.pl      |  1 -
 test_regress/t/t_lint_ifdepth_bad.pl       |  1 -
 test_regress/t/t_lint_implicit.pl          |  1 -
 test_regress/t/t_lint_implicit_bad.pl      |  1 -
 test_regress/t/t_lint_implicit_def_bad.pl  |  1 -
 test_regress/t/t_lint_implicit_port.pl     |  1 -
 test_regress/t/t_lint_importstar_bad.pl    |  1 -
 test_regress/t/t_lint_incabspath.pl        |  1 -
 test_regress/t/t_lint_incabspath_bad.pl    |  1 -
 test_regress/t/t_lint_latch_bad.pl         |  1 -
 test_regress/t/t_lint_subout_bad.pl        |  1 -
 test_regress/t/t_lint_syncasyncnet_bad.pl  |  1 -
 test_regress/t/t_lint_unused.pl            |  1 -
 test_regress/t/t_lint_unused_bad.pl        |  1 -
 test_regress/t/t_lint_unused_iface.pl      |  1 -
 test_regress/t/t_lint_unused_iface_bad.pl  |  1 -
 test_regress/t/t_math_sign_extend.v        |  1 -
 test_regress/t/t_param_func.v              |  1 -
 test_regress/t/t_param_sel_range.v         |  2 --
 test_regress/t/t_past_bad.pl               |  1 -
 test_regress/t/t_past_unsup_bad.pl         |  1 -
 test_regress/t/t_pp_circdef_bad.pl         |  1 -
 test_regress/t/t_pp_dupdef.pl              |  1 -
 test_regress/t/t_pp_dupdef_bad.pl          |  1 -
 test_regress/t/t_pp_lib.pl                 |  1 -
 test_regress/t/t_pp_misdef_bad.pl          |  1 -
 test_regress/t/t_pp_underline_bad.pl       |  1 -
 test_regress/t/t_preproc_persist_inc.v     |  1 -
 test_regress/t/t_select_bad_msb.pl         |  1 -
 test_regress/t/t_select_bad_range.pl       |  1 -
 test_regress/t/t_select_bad_range2.pl      |  1 -
 test_regress/t/t_select_bad_range3.pl      |  1 -
 test_regress/t/t_select_bad_tri.pl         |  1 -
 test_regress/t/t_select_index.pl           |  1 -
 test_regress/t/t_select_param.pl           |  1 -
 test_regress/t/t_select_runtime_range.pl   |  1 -
 test_regress/t/t_select_set.pl             |  1 -
 test_regress/t/t_struct_anon.v             |  1 -
 test_regress/t/t_struct_init.pl            |  1 -
 test_regress/t/t_struct_init_bad.pl        |  1 -
 test_regress/t/t_struct_nest.v             |  1 -
 test_regress/t/t_struct_pat_width.pl       |  1 -
 test_regress/t/t_struct_port.pl            |  1 -
 test_regress/t/t_sys_plusargs_bad.pl       |  1 -
 test_regress/t/t_sys_readmem_bad_digit.mem |  1 -
 test_regress/t/t_trace_fst.pl              |  1 -
 test_regress/t/t_tri_ifbegin.v             |  1 -
 test_regress/t/t_tri_inout2.v              |  1 -
 test_regress/t/t_tri_pull2_bad.pl          |  1 -
 test_regress/t/t_tri_pull_bad.pl           |  1 -
 test_regress/t/t_tri_various.v             |  1 -
 test_regress/t/t_typedef_array.v           |  1 -
 81 files changed, 14 insertions(+), 88 deletions(-)

diff --git a/src/V3Global.h b/src/V3Global.h
index b16fed088..b337b533e 100644
--- a/src/V3Global.h
+++ b/src/V3Global.h
@@ -125,5 +125,4 @@ extern V3Global v3Global;
 
 //######################################################################
 
-#endif // guard
-
+#endif  // guard
diff --git a/src/V3Hashed.cpp b/src/V3Hashed.cpp
index 699952809..ec26685f5 100644
--- a/src/V3Hashed.cpp
+++ b/src/V3Hashed.cpp
@@ -217,4 +217,3 @@ V3Hashed::iterator V3Hashed::findDuplicate(AstNode* nodep, V3HashedUserCheck* ch
     }
     return end();
 }
-
diff --git a/src/V3LanguageWords.h b/src/V3LanguageWords.h
index 9d058f660..b6b5fcf8b 100644
--- a/src/V3LanguageWords.h
+++ b/src/V3LanguageWords.h
@@ -186,5 +186,4 @@ class V3LanguageWords {
     }
 };
 
-#endif // Guard
-
+#endif  // Guard
diff --git a/src/verilog.l b/src/verilog.l
index 49242f0e0..f19865462 100644
--- a/src/verilog.l
+++ b/src/verilog.l
@@ -1127,4 +1127,3 @@ int V3ParseImp::lexToBison() {
     }
     return yylval.token;
 }
-
diff --git a/test_regress/input.vc b/test_regress/input.vc
index 6fb08696b..596545587 100644
--- a/test_regress/input.vc
+++ b/test_regress/input.vc
@@ -5,4 +5,3 @@
  +incdir+t
  +incdir+../include
  +incdir+obj_dir/
-
diff --git a/test_regress/t/t_bind2.v b/test_regress/t/t_bind2.v
index eae64c67b..b9cc73220 100644
--- a/test_regress/t/t_bind2.v
+++ b/test_regress/t/t_bind2.v
@@ -89,4 +89,3 @@ module mycheck (/*AUTOARG*/
       $finish;
    end
 endmodule
-
diff --git a/test_regress/t/t_case_dupitems.v b/test_regress/t/t_case_dupitems.v
index b5d6a9bab..c389a7ea4 100644
--- a/test_regress/t/t_case_dupitems.v
+++ b/test_regress/t/t_case_dupitems.v
@@ -78,4 +78,3 @@ module Test (/*AUTOARG*/
       endcase
    end
 endmodule
-
diff --git a/test_regress/t/t_case_reducer.v b/test_regress/t/t_case_reducer.v
index b96570a9d..306786154 100644
--- a/test_regress/t/t_case_reducer.v
+++ b/test_regress/t/t_case_reducer.v
@@ -276,4 +276,3 @@ module clz(
   assign out = {4'b0000, clz_byte1};
 
 endmodule // clz
-
diff --git a/test_regress/t/t_cellarray.v b/test_regress/t/t_cellarray.v
index 3be5db87b..6789ec737 100644
--- a/test_regress/t/t_cellarray.v
+++ b/test_regress/t/t_cellarray.v
@@ -97,4 +97,3 @@ module drv4
    assign datao = colSelB & colSelA & colSelC & colSelD;
 
 endmodule
-
diff --git a/test_regress/t/t_clk_inp_init.cpp b/test_regress/t/t_clk_inp_init.cpp
index 1f3f13991..62160bf8c 100644
--- a/test_regress/t/t_clk_inp_init.cpp
+++ b/test_regress/t/t_clk_inp_init.cpp
@@ -63,5 +63,3 @@ int main(int argc, char **argv, char **env) {
 
     return 0;
 }
-
-
diff --git a/test_regress/t/t_concat_opt.v b/test_regress/t/t_concat_opt.v
index 900a36f68..3eed6fafe 100644
--- a/test_regress/t/t_concat_opt.v
+++ b/test_regress/t/t_concat_opt.v
@@ -68,4 +68,3 @@ module t (/*AUTOARG*/
    end
 
 endmodule
-
diff --git a/test_regress/t/t_display_bad.pl b/test_regress/t/t_display_bad.pl
index fdfdab856..8bc867659 100755
--- a/test_regress/t/t_display_bad.pl
+++ b/test_regress/t/t_display_bad.pl
@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_dist_whitespace.pl b/test_regress/t/t_dist_whitespace.pl
index 2999b871f..5aecb910d 100755
--- a/test_regress/t/t_dist_whitespace.pl
+++ b/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";
+        }
     }
 }
 
diff --git a/test_regress/t/t_enum_public.v b/test_regress/t/t_enum_public.v
index f3b1a93e2..d26a77bb2 100644
--- a/test_regress/t/t_enum_public.v
+++ b/test_regress/t/t_enum_public.v
@@ -30,4 +30,3 @@ module t (/*AUTOARG*/);
    end
 
 endmodule
-
diff --git a/test_regress/t/t_flag_debug_noleak.v b/test_regress/t/t_flag_debug_noleak.v
index 92add3b4e..6b8661775 100644
--- a/test_regress/t/t_flag_debug_noleak.v
+++ b/test_regress/t/t_flag_debug_noleak.v
@@ -9,4 +9,3 @@ module t;
       $finish;
    end
 endmodule
-
diff --git a/test_regress/t/t_flag_f__2.vc b/test_regress/t/t_flag_f__2.vc
index 3ba204041..623d0992b 100644
--- a/test_regress/t/t_flag_f__2.vc
+++ b/test_regress/t/t_flag_f__2.vc
@@ -7,4 +7,3 @@
    */
 
 +define+GOT_DEF2=1
-
diff --git a/test_regress/t/t_flag_parameter.vc b/test_regress/t/t_flag_parameter.vc
index a51adbe3e..7946f2b28 100644
--- a/test_regress/t/t_flag_parameter.vc
+++ b/test_regress/t/t_flag_parameter.vc
@@ -12,4 +12,3 @@
 -pvalue+int32=123
 -Gint41=32'hdead_beef
 -pvalue+int42=32'hdead_beef
-
diff --git a/test_regress/t/t_flag_woff.v b/test_regress/t/t_flag_woff.v
index 6bef2db23..7c2936dd1 100644
--- a/test_regress/t/t_flag_woff.v
+++ b/test_regress/t/t_flag_woff.v
@@ -17,4 +17,3 @@ module t;
    wire C = 15'd1234;
 
 endmodule
-
diff --git a/test_regress/t/t_func_bad.pl b/test_regress/t/t_func_bad.pl
index fdfdab856..8bc867659 100755
--- a/test_regress/t/t_func_bad.pl
+++ b/test_regress/t/t_func_bad.pl
@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_func_bad2.pl b/test_regress/t/t_func_bad2.pl
index 37764365f..43933e6ae 100755
--- a/test_regress/t/t_func_bad2.pl
+++ b/test_regress/t/t_func_bad2.pl
@@ -16,4 +16,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_func_default_warn.v b/test_regress/t/t_func_default_warn.v
index e5748f813..3e555a093 100644
--- a/test_regress/t/t_func_default_warn.v
+++ b/test_regress/t/t_func_default_warn.v
@@ -25,4 +25,3 @@ module t (/*AUTOARG*/);
    end
 
 endmodule
-
diff --git a/test_regress/t/t_func_defaults.v b/test_regress/t/t_func_defaults.v
index daccafc92..5e3b60107 100644
--- a/test_regress/t/t_func_defaults.v
+++ b/test_regress/t/t_func_defaults.v
@@ -30,4 +30,3 @@ module t (/*AUTOARG*/);
    end
 
 endmodule
-
diff --git a/test_regress/t/t_gate_implicit.v b/test_regress/t/t_gate_implicit.v
index 5220f5c5e..4255ff293 100644
--- a/test_regress/t/t_gate_implicit.v
+++ b/test_regress/t/t_gate_implicit.v
@@ -77,4 +77,3 @@ endmodule
 module Mxor (output out, input a, b);
    assign out = (a ^ b);
 endmodule
-
diff --git a/test_regress/t/t_gen_div0.v b/test_regress/t/t_gen_div0.v
index fd6a9809d..2a552717a 100644
--- a/test_regress/t/t_gen_div0.v
+++ b/test_regress/t/t_gen_div0.v
@@ -36,4 +36,3 @@ module Test ();
         end
    endgenerate
 endmodule
-
diff --git a/test_regress/t/t_genvar_misuse_bad.v b/test_regress/t/t_genvar_misuse_bad.v
index 9a7239df7..7ab459216 100644
--- a/test_regress/t/t_genvar_misuse_bad.v
+++ b/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;
-   assign 	     q[i] = d[i];
+   genvar             i;
+   assign            q[i] = d[i];
 endmodule
-
diff --git a/test_regress/t/t_interface_missing_bad.v b/test_regress/t/t_interface_missing_bad.v
index d95134482..410c3c004 100644
--- a/test_regress/t/t_interface_missing_bad.v
+++ b/test_regress/t/t_interface_missing_bad.v
@@ -30,4 +30,3 @@ module t (/*AUTOARG*/);
    end
 
 endmodule
-
diff --git a/test_regress/t/t_lint_declfilename.pl b/test_regress/t/t_lint_declfilename.pl
index cf67c444c..369d69707 100755
--- a/test_regress/t/t_lint_declfilename.pl
+++ b/test_regress/t/t_lint_declfilename.pl
@@ -18,4 +18,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_declfilename_bad.pl b/test_regress/t/t_lint_declfilename_bad.pl
index 8a977b032..c9defd4c7 100755
--- a/test_regress/t/t_lint_declfilename_bad.pl
+++ b/test_regress/t/t_lint_declfilename_bad.pl
@@ -25,4 +25,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_defparam.pl b/test_regress/t/t_lint_defparam.pl
index cf67c444c..369d69707 100755
--- a/test_regress/t/t_lint_defparam.pl
+++ b/test_regress/t/t_lint_defparam.pl
@@ -18,4 +18,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_defparam_bad.pl b/test_regress/t/t_lint_defparam_bad.pl
index d67cfc4d1..079ba4535 100755
--- a/test_regress/t/t_lint_defparam_bad.pl
+++ b/test_regress/t/t_lint_defparam_bad.pl
@@ -25,4 +25,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_ifdepth_bad.pl b/test_regress/t/t_lint_ifdepth_bad.pl
index a1c7569ad..2ccfabd1d 100755
--- a/test_regress/t/t_lint_ifdepth_bad.pl
+++ b/test_regress/t/t_lint_ifdepth_bad.pl
@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_implicit.pl b/test_regress/t/t_lint_implicit.pl
index b757347fa..4288859cb 100755
--- a/test_regress/t/t_lint_implicit.pl
+++ b/test_regress/t/t_lint_implicit.pl
@@ -15,4 +15,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_implicit_bad.pl b/test_regress/t/t_lint_implicit_bad.pl
index 0971891d6..45b79cf32 100755
--- a/test_regress/t/t_lint_implicit_bad.pl
+++ b/test_regress/t/t_lint_implicit_bad.pl
@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_implicit_def_bad.pl b/test_regress/t/t_lint_implicit_def_bad.pl
index 827d54923..8a36606fe 100755
--- a/test_regress/t/t_lint_implicit_def_bad.pl
+++ b/test_regress/t/t_lint_implicit_def_bad.pl
@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_implicit_port.pl b/test_regress/t/t_lint_implicit_port.pl
index b132612a8..5c8e80b1f 100755
--- a/test_regress/t/t_lint_implicit_port.pl
+++ b/test_regress/t/t_lint_implicit_port.pl
@@ -15,4 +15,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_importstar_bad.pl b/test_regress/t/t_lint_importstar_bad.pl
index 413db0e11..7029a42d0 100755
--- a/test_regress/t/t_lint_importstar_bad.pl
+++ b/test_regress/t/t_lint_importstar_bad.pl
@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_incabspath.pl b/test_regress/t/t_lint_incabspath.pl
index decb90c06..160c9d393 100755
--- a/test_regress/t/t_lint_incabspath.pl
+++ b/test_regress/t/t_lint_incabspath.pl
@@ -18,4 +18,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_incabspath_bad.pl b/test_regress/t/t_lint_incabspath_bad.pl
index b5bafa4e5..63577bfa3 100755
--- a/test_regress/t/t_lint_incabspath_bad.pl
+++ b/test_regress/t/t_lint_incabspath_bad.pl
@@ -25,4 +25,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_latch_bad.pl b/test_regress/t/t_lint_latch_bad.pl
index 41457686b..9e07056fd 100755
--- a/test_regress/t/t_lint_latch_bad.pl
+++ b/test_regress/t/t_lint_latch_bad.pl
@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_subout_bad.pl b/test_regress/t/t_lint_subout_bad.pl
index fb4e1de76..076f15279 100755
--- a/test_regress/t/t_lint_subout_bad.pl
+++ b/test_regress/t/t_lint_subout_bad.pl
@@ -18,4 +18,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_syncasyncnet_bad.pl b/test_regress/t/t_lint_syncasyncnet_bad.pl
index a1c7569ad..2ccfabd1d 100755
--- a/test_regress/t/t_lint_syncasyncnet_bad.pl
+++ b/test_regress/t/t_lint_syncasyncnet_bad.pl
@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_unused.pl b/test_regress/t/t_lint_unused.pl
index 07151b701..b3ea3115d 100755
--- a/test_regress/t/t_lint_unused.pl
+++ b/test_regress/t/t_lint_unused.pl
@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_unused_bad.pl b/test_regress/t/t_lint_unused_bad.pl
index d90485d28..aa7bc6420 100755
--- a/test_regress/t/t_lint_unused_bad.pl
+++ b/test_regress/t/t_lint_unused_bad.pl
@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_unused_iface.pl b/test_regress/t/t_lint_unused_iface.pl
index 053e32067..7169b4873 100755
--- a/test_regress/t/t_lint_unused_iface.pl
+++ b/test_regress/t/t_lint_unused_iface.pl
@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_lint_unused_iface_bad.pl b/test_regress/t/t_lint_unused_iface_bad.pl
index e942e9b7b..169c414fa 100755
--- a/test_regress/t/t_lint_unused_iface_bad.pl
+++ b/test_regress/t/t_lint_unused_iface_bad.pl
@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_math_sign_extend.v b/test_regress/t/t_math_sign_extend.v
index 42d5d9a97..0dcba05c1 100644
--- a/test_regress/t/t_math_sign_extend.v
+++ b/test_regress/t/t_math_sign_extend.v
@@ -129,4 +129,3 @@ module t (/*AUTOARG*/
 
    end
 endmodule
-
diff --git a/test_regress/t/t_param_func.v b/test_regress/t/t_param_func.v
index 98fa4ed03..4706a8325 100644
--- a/test_regress/t/t_param_func.v
+++ b/test_regress/t/t_param_func.v
@@ -31,4 +31,3 @@ module test#(
     endfunction
 
 endmodule
-
diff --git a/test_regress/t/t_param_sel_range.v b/test_regress/t/t_param_sel_range.v
index ec8e69970..26bb6bed6 100644
--- a/test_regress/t/t_param_sel_range.v
+++ b/test_regress/t/t_param_sel_range.v
@@ -46,5 +46,3 @@ module submod (/*AUTOARG*/
    wire out = r_rst[STAGES-1];
 
 endmodule
-
-
diff --git a/test_regress/t/t_past_bad.pl b/test_regress/t/t_past_bad.pl
index 4f336495a..b3eb5b75d 100755
--- a/test_regress/t/t_past_bad.pl
+++ b/test_regress/t/t_past_bad.pl
@@ -16,4 +16,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_past_unsup_bad.pl b/test_regress/t/t_past_unsup_bad.pl
index e6683df33..519231f47 100755
--- a/test_regress/t/t_past_unsup_bad.pl
+++ b/test_regress/t/t_past_unsup_bad.pl
@@ -18,4 +18,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_pp_circdef_bad.pl b/test_regress/t/t_pp_circdef_bad.pl
index 28672314c..e5ca920e2 100755
--- a/test_regress/t/t_pp_circdef_bad.pl
+++ b/test_regress/t/t_pp_circdef_bad.pl
@@ -20,4 +20,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_pp_dupdef.pl b/test_regress/t/t_pp_dupdef.pl
index b9555f527..792632ceb 100755
--- a/test_regress/t/t_pp_dupdef.pl
+++ b/test_regress/t/t_pp_dupdef.pl
@@ -15,4 +15,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_pp_dupdef_bad.pl b/test_regress/t/t_pp_dupdef_bad.pl
index bf1d4b40e..08a437057 100755
--- a/test_regress/t/t_pp_dupdef_bad.pl
+++ b/test_regress/t/t_pp_dupdef_bad.pl
@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_pp_lib.pl b/test_regress/t/t_pp_lib.pl
index 9d820dc30..96c8d22f9 100755
--- a/test_regress/t/t_pp_lib.pl
+++ b/test_regress/t/t_pp_lib.pl
@@ -19,4 +19,3 @@ execute(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_pp_misdef_bad.pl b/test_regress/t/t_pp_misdef_bad.pl
index 1b89ea0fe..64051b030 100755
--- a/test_regress/t/t_pp_misdef_bad.pl
+++ b/test_regress/t/t_pp_misdef_bad.pl
@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_pp_underline_bad.pl b/test_regress/t/t_pp_underline_bad.pl
index 795815f25..47214f6e3 100755
--- a/test_regress/t/t_pp_underline_bad.pl
+++ b/test_regress/t/t_pp_underline_bad.pl
@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_preproc_persist_inc.v b/test_regress/t/t_preproc_persist_inc.v
index c7da7c2e9..0e62c6b1d 100644
--- a/test_regress/t/t_preproc_persist_inc.v
+++ b/test_regress/t/t_preproc_persist_inc.v
@@ -7,4 +7,3 @@
  `define COMMON_GUARD 1
 Inside `__FILE__.
 `endif
-
diff --git a/test_regress/t/t_select_bad_msb.pl b/test_regress/t/t_select_bad_msb.pl
index 8fe0627aa..96eb3a182 100755
--- a/test_regress/t/t_select_bad_msb.pl
+++ b/test_regress/t/t_select_bad_msb.pl
@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_select_bad_range.pl b/test_regress/t/t_select_bad_range.pl
index 8bfbcfc8b..96456db03 100755
--- a/test_regress/t/t_select_bad_range.pl
+++ b/test_regress/t/t_select_bad_range.pl
@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_select_bad_range2.pl b/test_regress/t/t_select_bad_range2.pl
index 8bfbcfc8b..96456db03 100755
--- a/test_regress/t/t_select_bad_range2.pl
+++ b/test_regress/t/t_select_bad_range2.pl
@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_select_bad_range3.pl b/test_regress/t/t_select_bad_range3.pl
index 8bfbcfc8b..96456db03 100755
--- a/test_regress/t/t_select_bad_range3.pl
+++ b/test_regress/t/t_select_bad_range3.pl
@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_select_bad_tri.pl b/test_regress/t/t_select_bad_tri.pl
index 8bfbcfc8b..96456db03 100755
--- a/test_regress/t/t_select_bad_tri.pl
+++ b/test_regress/t/t_select_bad_tri.pl
@@ -17,4 +17,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_select_index.pl b/test_regress/t/t_select_index.pl
index c9adf6d32..89a4e77d9 100755
--- a/test_regress/t/t_select_index.pl
+++ b/test_regress/t/t_select_index.pl
@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_select_param.pl b/test_regress/t/t_select_param.pl
index c9adf6d32..89a4e77d9 100755
--- a/test_regress/t/t_select_param.pl
+++ b/test_regress/t/t_select_param.pl
@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_select_runtime_range.pl b/test_regress/t/t_select_runtime_range.pl
index c9adf6d32..89a4e77d9 100755
--- a/test_regress/t/t_select_runtime_range.pl
+++ b/test_regress/t/t_select_runtime_range.pl
@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_select_set.pl b/test_regress/t/t_select_set.pl
index c9adf6d32..89a4e77d9 100755
--- a/test_regress/t/t_select_set.pl
+++ b/test_regress/t/t_select_set.pl
@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_struct_anon.v b/test_regress/t/t_struct_anon.v
index 2b97364bc..2287a589c 100644
--- a/test_regress/t/t_struct_anon.v
+++ b/test_regress/t/t_struct_anon.v
@@ -23,4 +23,3 @@ module t (
    initial struct2 = 64'h123456789_abcdef0;
    always_comb s2 = struct2;
 endmodule
-
diff --git a/test_regress/t/t_struct_init.pl b/test_regress/t/t_struct_init.pl
index c9adf6d32..89a4e77d9 100755
--- a/test_regress/t/t_struct_init.pl
+++ b/test_regress/t/t_struct_init.pl
@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_struct_init_bad.pl b/test_regress/t/t_struct_init_bad.pl
index 43c15946f..5fe79d7c0 100755
--- a/test_regress/t/t_struct_init_bad.pl
+++ b/test_regress/t/t_struct_init_bad.pl
@@ -19,4 +19,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_struct_nest.v b/test_regress/t/t_struct_nest.v
index 33790b832..9059568f3 100644
--- a/test_regress/t/t_struct_nest.v
+++ b/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
-
diff --git a/test_regress/t/t_struct_pat_width.pl b/test_regress/t/t_struct_pat_width.pl
index c9adf6d32..89a4e77d9 100755
--- a/test_regress/t/t_struct_pat_width.pl
+++ b/test_regress/t/t_struct_pat_width.pl
@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_struct_port.pl b/test_regress/t/t_struct_port.pl
index c9adf6d32..89a4e77d9 100755
--- a/test_regress/t/t_struct_port.pl
+++ b/test_regress/t/t_struct_port.pl
@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_sys_plusargs_bad.pl b/test_regress/t/t_sys_plusargs_bad.pl
index c9adf6d32..89a4e77d9 100755
--- a/test_regress/t/t_sys_plusargs_bad.pl
+++ b/test_regress/t/t_sys_plusargs_bad.pl
@@ -18,4 +18,3 @@ execute(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_sys_readmem_bad_digit.mem b/test_regress/t/t_sys_readmem_bad_digit.mem
index 8e17173a0..bd4ba9b2d 100644
--- a/test_regress/t/t_sys_readmem_bad_digit.mem
+++ b/test_regress/t/t_sys_readmem_bad_digit.mem
@@ -6,4 +6,3 @@
 // Version 2.0.
 
 a0
-
diff --git a/test_regress/t/t_trace_fst.pl b/test_regress/t/t_trace_fst.pl
index a8dfc3cca..54dde9646 100755
--- a/test_regress/t/t_trace_fst.pl
+++ b/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;
-
diff --git a/test_regress/t/t_tri_ifbegin.v b/test_regress/t/t_tri_ifbegin.v
index 899d051df..c48467ff6 100644
--- a/test_regress/t/t_tri_ifbegin.v
+++ b/test_regress/t/t_tri_ifbegin.v
@@ -49,4 +49,3 @@ module sub (/*AUTOARG*/
    endgenerate
 
 endmodule
-
diff --git a/test_regress/t/t_tri_inout2.v b/test_regress/t/t_tri_inout2.v
index 4c9512cdc..a7b66982e 100644
--- a/test_regress/t/t_tri_inout2.v
+++ b/test_regress/t/t_tri_inout2.v
@@ -73,4 +73,3 @@ endmodule
 module ChildB(input A, output Y);
   assign Y = A;
 endmodule
-
diff --git a/test_regress/t/t_tri_pull2_bad.pl b/test_regress/t/t_tri_pull2_bad.pl
index 37764365f..43933e6ae 100755
--- a/test_regress/t/t_tri_pull2_bad.pl
+++ b/test_regress/t/t_tri_pull2_bad.pl
@@ -16,4 +16,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_tri_pull_bad.pl b/test_regress/t/t_tri_pull_bad.pl
index 37764365f..43933e6ae 100755
--- a/test_regress/t/t_tri_pull_bad.pl
+++ b/test_regress/t/t_tri_pull_bad.pl
@@ -16,4 +16,3 @@ compile(
 
 ok(1);
 1;
-
diff --git a/test_regress/t/t_tri_various.v b/test_regress/t/t_tri_various.v
index 8a3367298..945130d70 100644
--- a/test_regress/t/t_tri_various.v
+++ b/test_regress/t/t_tri_various.v
@@ -208,4 +208,3 @@ endmodule
 //      end
 //   end
 //endmodule
-
diff --git a/test_regress/t/t_typedef_array.v b/test_regress/t/t_typedef_array.v
index 1cc4c44e5..cabcf3b2e 100644
--- a/test_regress/t/t_typedef_array.v
+++ b/test_regress/t/t_typedef_array.v
@@ -23,4 +23,3 @@ module t;
       $finish;
    end
 endmodule
-