minor cleanup

Author: dchakarwarti

proto_processing_lib/proto_scrubber/cloud_audit_log_field_checker.h

@@ -50,6 +50,8 @@ class CloudAuditLogFieldChecker
   // and "c", and the message "a" can include only field "b".
   absl::Status AddOrIntersectFieldPaths(
       const std::vector<std::string>& paths) override;
+
+  using CloudAuditLogFieldCheckerInterface::CheckField;
   // Decides whether the given field path is included, partially included, or
   // excluded.
   //

proto_processing_lib/proto_scrubber/proto_scrubber.cc

@@ -727,7 +727,6 @@ absl::StatusOr<uint32_t> ProtoScrubber::ScanMapEntry(
   int current_position = start_position;
   bool is_key_found = false;
   bool is_value_added = false;
-  bool is_value_null = true;
   FieldFilters excluded_by_filter = FieldFilters::Unspecified;
   uint32_t tag = input_stream->ReadTag();
   while (tag != 0) {
@@ -761,7 +760,6 @@ absl::StatusOr<uint32_t> ProtoScrubber::ScanMapEntry(
         }
       }
     } else if (field_number == 2) {
-      is_value_null = false;
       if (check_result == FieldCheckResults::kExclude) {
         IncreaseFieldDroppedCount(map_type.name(), field_number,
                                   scrubber_context_, excluded_by_filter);

proto_processing_lib/proto_scrubber/proto_scrubber.h

@@ -84,6 +84,7 @@ class ProtoScrubber : public ProtoScrubberInterface {
   virtual absl::StatusOr<uint32_t> ScrubWithChecksum(
       google::protobuf::field_extraction::MessageData* message) const override;
 
+  using ProtoScrubberInterface::CheckField;
   // Checks if a field should be kept by calling field_checkers_ and returns the
   // aggregated result. If one of them returns EXCLUDE, the aggregated result is
   // EXCLUDE. If all of them return INCLUDE, the aggregated result is INCLUDE.