Revise and enable test for hf[] FT_BOOLEAN fields with a non-zero bit mask and with...
authorBill Meier <wmeier@newsguy.com>
Wed, 7 Mar 2012 16:19:24 +0000 (16:19 -0000)
committerBill Meier <wmeier@newsguy.com>
Wed, 7 Mar 2012 16:19:24 +0000 (16:19 -0000)
svn path=/trunk/; revision=41399

tools/checkAPIs.pl

index f43cc966450b08b3ada9fceb0ed6141a5906c118..a31f3d5be6da7f2d218fb3c1b800784a90ef608a 100755 (executable)
@@ -1572,15 +1572,16 @@ sub check_hf_entries($$)
                         print STDERR "Error: $abbrev uses a value_string but is an FT_BOOLEAN in $filename\n";
                         $errorCount++;
                }
+               if (($ft eq "FT_BOOLEAN") && ($bitmask !~ /^(0x)?0+$/) && ($display =~ /^BASE_/)) {
+                       print STDERR "Error: $abbrev: FT_BOOLEAN with a bitmask must specify a 'parent field width' for 'display' in $filename\n";
+                       $errorCount++;
+               }
+## Benign...
 ##             if (($ft eq "FT_BOOLEAN") && ($bitmask =~ /^(0x)?0+$/) && ($display ne "BASE_NONE")) {
-##                     print STDERR "Error: $abbrev FT_BOOLEAN with no bitmask must use BASE_NONE for 'display' in $filename\n";
-##                        $errorCount++;
-##                }
-##             if (($ft eq "FT_BOOLEAN") && ($bitmask !~ /^(0x)?0+$/) && ($display !~ /^\d+$/)) {
-##                     print STDERR "Error: $abbrev FT_BOOLEAN with a bitmask must use a numeric vale for 'display' in $filename\n";
-##                        $errorCount++;
-##                }
-        }
+##                     print STDERR "Error: $abbrev: FT_BOOLEAN with no bitmask must use BASE_NONE for 'display' in $filename\n";
+##                     $errorCount++;
+##             }
+       }
 
         return $errorCount;
 }