Replace kind variable from validator API with Boolean stubout parameter.

src/tools/validator_tools/ncstubout.c

Index: src/tools/validator_tools/ncstubout.c
diff --git a/src/tools/validator_tools/ncstubout.c b/src/tools/validator_tools/ncstubout.c
index f167df8c7629ef17cc2faadaa0791d89f2c412a8..ac00a2b308a52e5a7dd3b57080b81a201334248b 100644
--- a/src/tools/validator_tools/ncstubout.c
+++ b/src/tools/validator_tools/ncstubout.c
@@ -39,8 +39,8 @@ static Bool FixUpSection(uintptr_t load_address,
   status = NACL_SUBARCH_NAME(ApplyValidator,
                              NACL_TARGET_ARCH,
                              NACL_TARGET_SUBARCH)
-      (sb_kind, NaClApplyValidationDoStubout, load_address, code,
-       code_size, bundle_size, FALSE, &cpu_features, NULL);
+      (sb_kind, load_address, code, code_size, bundle_size, TRUE, FALSE,

[Mark Seaborn, 2012/04/10 00:57:38]

Can you write it as
/* stubout_mode= */ TRUE,
/* readonly_text= */ FALSE
for readability?

[Nick Bray (chromium), 2012/04/10 18:49:20]

Done.

+       &cpu_features, NULL);
   if (status == NaClValidationSucceeded) {
     /* Now run the validator again, so that we report any errors
      * that were not fixed by stubbing out. This is done so that
@@ -49,8 +49,7 @@ static Bool FixUpSection(uintptr_t load_address,
     status = NACL_SUBARCH_NAME(ApplyValidatorVerbosely,
                                NACL_TARGET_ARCH,
                                NACL_TARGET_SUBARCH)
-        (sb_kind, NaClApplyCodeValidation, load_address, code, code_size,
-         bundle_size, &cpu_features);
+        (sb_kind, load_address, code, code_size, bundle_size, &cpu_features);
   }
 
   switch (status) {