diff options
author | Jeff Morriss <jeff.morriss@ulticom.com> | 2011-01-04 18:43:24 +0000 |
---|---|---|
committer | Jeff Morriss <jeff.morriss@ulticom.com> | 2011-01-04 18:43:24 +0000 |
commit | c18d15f9f7fe6913c628fe95e90d630eb0f4c796 (patch) | |
tree | 483428d2080d8b425331608207f9b6cbe57a84c1 /tools/checkAPIs.pl | |
parent | f94b75c6582ec292b63f253af5ae54933c4ff17c (diff) | |
download | wireshark-c18d15f9f7fe6913c628fe95e90d630eb0f4c796.tar.gz |
Detect (and error on) NULL or empty abbreviations. Wireshark aborts on the former and warns on the latter.
svn path=/trunk/; revision=35363
Diffstat (limited to 'tools/checkAPIs.pl')
-rwxr-xr-x | tools/checkAPIs.pl | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tools/checkAPIs.pl b/tools/checkAPIs.pl index e8787ca8cf..de661f77c1 100755 --- a/tools/checkAPIs.pl +++ b/tools/checkAPIs.pl @@ -978,7 +978,7 @@ sub check_hf_entries($$) \{\s* ("[A-Z0-9 '\./\(\)_:-]+") # name \s*,\s* - ("[A-Z0-9_\.-]+") # abbrev + (NULL|"[A-Z0-9_\.-]*") # abbrev \s*,\s* (FT_[A-Z0-9_]+) # field type \s*,\s* @@ -1000,6 +1000,10 @@ sub check_hf_entries($$) #print "name=$name, abbrev=$abbrev, ft=$ft, display=$display, convert=$convert, bitmask=$bitmask, blurb=$blurb\n"; + if ($abbrev eq '""' || $abbrev eq "NULL") { + print STDERR "Error: field $name does not have an abbreviation in $filename\n"; + $errorCount++; + } if ($name eq $abbrev) { print STDERR "Error: the abbreviation for field $name matches the field name in $filename\n"; $errorCount++; |