Removing unused variable in function declaration

Author: dchakarwarti

proto_processing_lib/proto_scrubber/field_checker_interface.h

@@ -68,7 +68,7 @@ class FieldCheckerInterface {
   // Can be used for stack overflow protection by each field checker.
   virtual FieldCheckResults CheckField(const std::vector<std::string>& path,
                                        const google::protobuf::Field* field,
-                                       const int field_depth) const {
+                                       const int /*field_depth*/) const {
     return CheckField(path, field);
   }
 
@@ -77,7 +77,7 @@ class FieldCheckerInterface {
   virtual FieldCheckResults CheckField(
       const std::vector<std::string>& path,
       const google::protobuf::Field* field, const int field_depth,
-      const google::protobuf::Type* parent_type) const {
+      const google::protobuf::Type* /*parent_type*/) const {
     return CheckField(path, field, field_depth);
   }
 

proto_processing_lib/proto_scrubber/field_mask_node.cc

@@ -227,7 +227,7 @@ absl::Status FieldMaskNode::InsertValueField(
   return AddChild(value, type_, "", false, next, end);
 }
 
-const bool FieldMaskNode::HasChild(absl::string_view field_name) const {
+bool FieldMaskNode::HasChild(absl::string_view field_name) const {
   return children_.contains(field_name);
 }
 

proto_processing_lib/proto_scrubber/field_mask_node.h

@@ -82,7 +82,7 @@ class FieldMaskNode {
   std::unique_ptr<FieldMaskNode> Intersect(const FieldMaskNode& other) const;
 
   // Whether there is a child node with given field name.
-  const bool HasChild(absl::string_view field_name) const;
+  bool HasChild(absl::string_view field_name) const;
 
   // Finds the child node for the given field name. Returns nullptr if no
   // children exist for the field name.