Use exactly one of final, override, and virtual

util/mach/exc_server_variants.h

 // Copyright 2014 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 85 matching lines @@
         bool* destroy_request) = 0;
 
    protected:
     ~Interface() {}
   };
 
   explicit ExcServer(Interface* interface);
 
   // MachMessageServer::Interface:
 
-  virtual bool MachMessageServerFunction(
-      const mach_msg_header_t* in_header,
-      mach_msg_header_t* out_header,
-      bool* destroy_complex_request) override;
+  bool MachMessageServerFunction(const mach_msg_header_t* in_header,
+                                 mach_msg_header_t* out_header,
+                                 bool* destroy_complex_request) override;
 
-  virtual mach_msg_size_t MachMessageServerRequestSize() override;
-  virtual mach_msg_size_t MachMessageServerReplySize() override;
+  mach_msg_size_t MachMessageServerRequestSize() override;
+  mach_msg_size_t MachMessageServerReplySize() override;
 
  private:
   Interface* interface_;  // weak
 };
 
 //! \brief A server interface for the `mach_exc` Mach subsystem.
 class MachExcServer : public MachMessageServer::Interface {
  public:
   //! \brief An interface that the different request messages that are a part of
   //!     the `mach_exc` Mach subsystem can be dispatched to.
@@ skipping 61 matching lines @@
         bool* destroy_request) = 0;
 
    protected:
     ~Interface() {}
   };
 
   explicit MachExcServer(Interface* interface);
 
   // MachMessageServer::Interface:
 
-  virtual bool MachMessageServerFunction(
-      const mach_msg_header_t* in_header,
-      mach_msg_header_t* out_header,
-      bool* destroy_complex_request) override;
+  bool MachMessageServerFunction(const mach_msg_header_t* in_header,
+                                 mach_msg_header_t* out_header,
+                                 bool* destroy_complex_request) override;
 
-  virtual mach_msg_size_t MachMessageServerRequestSize() override;
-  virtual mach_msg_size_t MachMessageServerReplySize() override;
+  mach_msg_size_t MachMessageServerRequestSize() override;
+  mach_msg_size_t MachMessageServerReplySize() override;
 
  private:
   Interface* interface_;  // weak
 };
 
 //! \brief A server interface for the `exc` Mach subsystem, simplified to have
 //!     only a single interface method needing implementation.
 class SimplifiedExcServer : public ExcServer, public ExcServer::Interface {
  public:
   //! \brief An interface that the different request messages that are a part of
@@ skipping 31 matching lines @@
         bool* destroy_complex_request) = 0;
 
    protected:
     ~Interface() {}
   };
 
   explicit SimplifiedExcServer(Interface* interface);
 
   // ExcServer::Interface:
 
-  virtual kern_return_t CatchExceptionRaise(exception_handler_t exception_port,
-                                            thread_t thread,
-                                            task_t task,
-                                            exception_type_t exception,
-                                            const exception_data_type_t* code,
-                                            mach_msg_type_number_t code_count,
-                                            bool* destroy_request) override;
-  virtual kern_return_t CatchExceptionRaiseState(
+  kern_return_t CatchExceptionRaise(exception_handler_t exception_port,
+                                    thread_t thread,
+                                    task_t task,
+                                    exception_type_t exception,
+                                    const exception_data_type_t* code,
+                                    mach_msg_type_number_t code_count,
+                                    bool* destroy_request) override;
+  kern_return_t CatchExceptionRaiseState(
       exception_handler_t exception_port,
       exception_type_t exception,
       const exception_data_type_t* code,
       mach_msg_type_number_t code_count,
       thread_state_flavor_t* flavor,
       const natural_t* old_state,
       mach_msg_type_number_t old_state_count,
       thread_state_t new_state,
       mach_msg_type_number_t* new_state_count) override;
-  virtual kern_return_t CatchExceptionRaiseStateIdentity(
+  kern_return_t CatchExceptionRaiseStateIdentity(
       exception_handler_t exception_port,
       thread_t thread,
       task_t task,
       exception_type_t exception,
       const exception_data_type_t* code,
       mach_msg_type_number_t code_count,
       thread_state_flavor_t* flavor,
       const natural_t* old_state,
       mach_msg_type_number_t old_state_count,
       thread_state_t new_state,
@@ skipping 53 matching lines @@
         bool* destroy_complex_request) = 0;
 
    protected:
     ~Interface() {}
   };
 
   explicit SimplifiedMachExcServer(Interface* interface);
 
   // MachExcServer::Interface:
 
-  virtual kern_return_t CatchMachExceptionRaise(
-      exception_handler_t exception_port,
-      thread_t thread,
-      task_t task,
-      exception_type_t exception,
-      const mach_exception_data_type_t* code,
-      mach_msg_type_number_t code_count,
-      bool* destroy_request) override;
-  virtual kern_return_t CatchMachExceptionRaiseState(
+  kern_return_t CatchMachExceptionRaise(exception_handler_t exception_port,
+                                        thread_t thread,
+                                        task_t task,
+                                        exception_type_t exception,
+                                        const mach_exception_data_type_t* code,
+                                        mach_msg_type_number_t code_count,
+                                        bool* destroy_request) override;
+  kern_return_t CatchMachExceptionRaiseState(
       exception_handler_t exception_port,
       exception_type_t exception,
       const mach_exception_data_type_t* code,
       mach_msg_type_number_t code_count,
       thread_state_flavor_t* flavor,
       const natural_t* old_state,
       mach_msg_type_number_t old_state_count,
       thread_state_t new_state,
       mach_msg_type_number_t* new_state_count) override;
-  virtual kern_return_t CatchMachExceptionRaiseStateIdentity(
+  kern_return_t CatchMachExceptionRaiseStateIdentity(
       exception_handler_t exception_port,
       thread_t thread,
       task_t task,
       exception_type_t exception,
       const mach_exception_data_type_t* code,
       mach_msg_type_number_t code_count,
       thread_state_flavor_t* flavor,
       const natural_t* old_state,
       mach_msg_type_number_t old_state_count,
       thread_state_t new_state,
@@ skipping 21 matching lines @@
 //! implementations.
 class UniversalMachExcServer
     : public MachMessageServer::Interface,
       public internal::SimplifiedExcServer::Interface,
       public internal::SimplifiedMachExcServer::Interface {
  public:
   UniversalMachExcServer();
 
   // MachMessageServer::Interface:
 
-  virtual bool MachMessageServerFunction(
-      const mach_msg_header_t* in_header,
-      mach_msg_header_t* out_header,
-      bool* destroy_complex_request) override;
+  bool MachMessageServerFunction(const mach_msg_header_t* in_header,
+                                 mach_msg_header_t* out_header,
+                                 bool* destroy_complex_request) override;
 
-  virtual mach_msg_size_t MachMessageServerRequestSize() override;
-  virtual mach_msg_size_t MachMessageServerReplySize() override;
+  mach_msg_size_t MachMessageServerRequestSize() override;
+  mach_msg_size_t MachMessageServerReplySize() override;
 
   // internal::SimplifiedExcServer::Interface:
 
-  virtual kern_return_t CatchException(exception_behavior_t behavior,
-                                       exception_handler_t exception_port,
-                                       thread_t thread,
-                                       task_t task,
-                                       exception_type_t exception,
-                                       const exception_data_type_t* code,
-                                       mach_msg_type_number_t code_count,
-                                       thread_state_flavor_t* flavor,
-                                       const natural_t* old_state,
-                                       mach_msg_type_number_t old_state_count,
-                                       thread_state_t new_state,
-                                       mach_msg_type_number_t* new_state_count,
-                                       bool* destroy_complex_request) override;
+  kern_return_t CatchException(exception_behavior_t behavior,
+                               exception_handler_t exception_port,
+                               thread_t thread,
+                               task_t task,
+                               exception_type_t exception,
+                               const exception_data_type_t* code,
+                               mach_msg_type_number_t code_count,
+                               thread_state_flavor_t* flavor,
+                               const natural_t* old_state,
+                               mach_msg_type_number_t old_state_count,
+                               thread_state_t new_state,
+                               mach_msg_type_number_t* new_state_count,
+                               bool* destroy_complex_request) override;
 
  private:
   internal::SimplifiedExcServer exc_server_;
   internal::SimplifiedMachExcServer mach_exc_server_;
 };
 
 //! \brief Recovers the original exception, first exception code, and signal
 //!     from the encoded form of the first exception code delivered with
 //!     `EXC_CRASH` exceptions.
 //!
@@ skipping 73 matching lines @@
 //!     \a behavior is not a state-carrying behavior, or when it is a
 //!     state-carrying behavior and \a set_thread_state is `true`.
 //!     `MACH_RCV_PORT_DIED` is used when \a behavior is a state-carrying
 //!     behavior and \a set_thread_state is `false`.
 kern_return_t ExcServerSuccessfulReturnValue(exception_behavior_t behavior,
                                              bool set_thread_state);
 
 }  // namespace crashpad
 
 #endif  // CRASHPAD_UTIL_MACH_EXC_SERVER_VARIANTS_H_