diff options
author | Peter Dillinger <peterd@meta.com> | 2024-01-31 13:14:42 -0800 |
---|---|---|
committer | Facebook GitHub Bot <facebook-github-bot@users.noreply.github.com> | 2024-01-31 13:14:42 -0800 |
commit | 76c834e441261662bc3af58551fd1bb03c8b1d39 (patch) | |
tree | e72bc8037d98b48cfb825d774eeec78115bda297 /port/win | |
parent | 95d582e0cc352033661efc232e41fc46e5c25dd3 (diff) |
Remove 'virtual' when implied by 'override' (#12319)
Summary:
... to follow modern C++ style / idioms.
Used this hack:
```
for FILE in `cat my_list_of_files`; do perl -pi -e 'BEGIN{undef $/;} s/ virtual( [^;{]* override)/$1/smg' $FILE; done
```
Pull Request resolved: https://github.com/facebook/rocksdb/pull/12319
Test Plan: existing tests, CI
Reviewed By: jaykorean
Differential Revision: D53275303
Pulled By: pdillinger
fbshipit-source-id: bc0881af270aa8ef4d0ae4f44c5a6614b6407377
Diffstat (limited to 'port/win')
-rw-r--r-- | port/win/io_win.h | 30 |
1 files changed, 13 insertions, 17 deletions
diff --git a/port/win/io_win.h b/port/win/io_win.h index a4fee8346..e1a6197ce 100644 --- a/port/win/io_win.h +++ b/port/win/io_win.h @@ -125,9 +125,7 @@ class WinSequentialFile : protected WinFileData, public FSSequentialFile { IOStatus InvalidateCache(size_t offset, size_t length) override; - virtual bool use_direct_io() const override { - return WinFileData::use_direct_io(); - } + bool use_direct_io() const override { return WinFileData::use_direct_io(); } }; // mmap() based random-access @@ -151,9 +149,9 @@ class WinMmapReadableFile : private WinFileData, public FSRandomAccessFile { Slice* result, char* scratch, IODebugContext* dbg) const override; - virtual IOStatus InvalidateCache(size_t offset, size_t length) override; + IOStatus InvalidateCache(size_t offset, size_t length) override; - virtual size_t GetUniqueId(char* id, size_t max_size) const override; + size_t GetUniqueId(char* id, size_t max_size) const override; }; // We preallocate and use memcpy to append new @@ -243,7 +241,7 @@ class WinMmapFile : private WinFileData, public FSWritableFile { IOStatus Allocate(uint64_t offset, uint64_t len, const IOOptions& options, IODebugContext* dbg) override; - virtual size_t GetUniqueId(char* id, size_t max_size) const override; + size_t GetUniqueId(char* id, size_t max_size) const override; }; class WinRandomAccessImpl { @@ -287,15 +285,13 @@ class WinRandomAccessFile Slice* result, char* scratch, IODebugContext* dbg) const override; - virtual size_t GetUniqueId(char* id, size_t max_size) const override; + size_t GetUniqueId(char* id, size_t max_size) const override; - virtual bool use_direct_io() const override { - return WinFileData::use_direct_io(); - } + bool use_direct_io() const override { return WinFileData::use_direct_io(); } IOStatus InvalidateCache(size_t offset, size_t length) override; - virtual size_t GetRequiredBufferAlignment() const override; + size_t GetRequiredBufferAlignment() const override; }; // This is a sequential write class. It has been mimicked (as others) after @@ -399,20 +395,20 @@ class WinWritableFile : private WinFileData, IOStatus Fsync(const IOOptions& options, IODebugContext* dbg) override; - virtual bool IsSyncThreadSafe() const override; + bool IsSyncThreadSafe() const override; // Indicates if the class makes use of direct I/O // Use PositionedAppend - virtual bool use_direct_io() const override; + bool use_direct_io() const override; - virtual size_t GetRequiredBufferAlignment() const override; + size_t GetRequiredBufferAlignment() const override; uint64_t GetFileSize(const IOOptions& options, IODebugContext* dbg) override; IOStatus Allocate(uint64_t offset, uint64_t len, const IOOptions& options, IODebugContext* dbg) override; - virtual size_t GetUniqueId(char* id, size_t max_size) const override; + size_t GetUniqueId(char* id, size_t max_size) const override; }; class WinRandomRWFile : private WinFileData, @@ -427,11 +423,11 @@ class WinRandomRWFile : private WinFileData, // Indicates if the class makes use of direct I/O // If false you must pass aligned buffer to Write() - virtual bool use_direct_io() const override; + bool use_direct_io() const override; // Use the returned alignment value to allocate aligned // buffer for Write() when use_direct_io() returns true - virtual size_t GetRequiredBufferAlignment() const override; + size_t GetRequiredBufferAlignment() const override; // Write bytes in `data` at offset `offset`, Returns Status::OK() on success. // Pass aligned buffer when use_direct_io() returns true. |