From 50614a95baeeee6b5ecc71cfdbdb655ade5bb2d3 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Jo=C3=A3o=20Valverde?= Date: Sun, 3 Jul 2016 13:17:41 +0100 Subject: [PATCH] checkhf: Use colon as field separator for better readability MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Comma looks like a list when skimming quickly. Change-Id: Ic0ac1bc8746c70724d2279e3d725c5c9449b79d8 Reviewed-on: https://code.wireshark.org/review/16260 Reviewed-by: João Valverde Reviewed-by: Michael Mann --- tools/checkhf.pl | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/checkhf.pl b/tools/checkhf.pl index 74b7bddf5c..27d51d3c05 100755 --- a/tools/checkhf.pl +++ b/tools/checkhf.pl @@ -135,8 +135,8 @@ while (my $filename = $ARGV[0]) { $unused_ei = diff_hash(\%ei_static_defs, \%ei_usage); - print_list("Unused href entry: $filename, ", $unused_href); - print_list("Unused ei entry: $filename, ", $unused_ei); + print_list("Unused href entry: $filename: ", $unused_href); + print_list("Unused ei entry: $filename: ", $unused_ei); # 2. Are all the hf_defs and ei_ entries (static and global) in [hf|ei]_array_entries ? # (Note: if a static hf_def or ei is "unused", don't check for same in [hf|ei]_array_entries) @@ -150,8 +150,8 @@ while (my $filename = $ARGV[0]) { $no_array_ei = diff_hash(\%ei_static_defs, \%ei_array_entries); $no_array_ei = diff_hash($no_array_ei, $unused_ei); # Remove "unused" ei_... from no_array list - print_list("ERROR: NO ARRAY: $filename, ", $no_array_href); - print_list("ERROR: NO ARRAY: $filename, ", $no_array_ei); + print_list("ERROR: NO ARRAY: $filename: ", $no_array_href); + print_list("ERROR: NO ARRAY: $filename: ", $no_array_ei); if ((keys %{$no_array_href}) != 0) { $error += 1; -- 2.34.1