[Lldb-commits] [lldb] r229084 - Fix four missing 'override' specifiers found with the Clang

Chandler Carruth chandlerc at gmail.com
Fri Feb 13 00:07:01 PST 2015


Author: chandlerc
Date: Fri Feb 13 02:07:00 2015
New Revision: 229084

URL: http://llvm.org/viewvc/llvm-project?rev=229084&view=rev
Log:
Fix four missing 'override' specifiers found with the Clang
'-Winconsistent-missing-override' warning. I suggest folks use this to
ensure that override is consistently used to mark virtual function
overrides.

Modified:
    lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h

Modified: lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h?rev=229084&r1=229083&r2=229084&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h (original)
+++ lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h Fri Feb 13 02:07:00 2015
@@ -49,10 +49,10 @@ namespace lldb_private
         IsWatchpointVacant(uint32_t wp_index);
 
         bool
-        ClearHardwareWatchpoint(uint32_t wp_index);
+        ClearHardwareWatchpoint(uint32_t wp_index) override;
 
         Error
-        ClearAllHardwareWatchpoints ();
+        ClearAllHardwareWatchpoints () override;
 
         Error
         SetHardwareWatchpointWithIndex(lldb::addr_t addr, size_t size,
@@ -60,13 +60,13 @@ namespace lldb_private
 
         uint32_t
         SetHardwareWatchpoint(lldb::addr_t addr, size_t size,
-                uint32_t watch_flags);
+                uint32_t watch_flags) override;
 
         lldb::addr_t
         GetWatchpointAddress(uint32_t wp_index);
 
         uint32_t
-        NumSupportedHardwareWatchpoints();
+        NumSupportedHardwareWatchpoints() override;
 
     private:
 





More information about the lldb-commits mailing list