diff options
Diffstat (limited to 'absl/flags/internal/commandlineflag.h')
| -rw-r--r-- | absl/flags/internal/commandlineflag.h | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/absl/flags/internal/commandlineflag.h b/absl/flags/internal/commandlineflag.h index 572568b93711..284286b6be18 100644 --- a/absl/flags/internal/commandlineflag.h +++ b/absl/flags/internal/commandlineflag.h @@ -207,7 +207,6 @@ class CommandLineFlag { def_(def), cur_(cur), counter_(0), - atomic_(kAtomicInit), locks_(nullptr) {} // Virtual destructor @@ -235,6 +234,10 @@ class CommandLineFlag { std::string DefaultValue() const; std::string CurrentValue() const; + // Interface to store the value in atomic if one used. This is short term + // interface. To be reworked once cur_ is moved. + virtual void StoreAtomic() {} + // Interfaces to operate on validators. virtual bool HasValidatorFn() const { return false; } virtual bool InvokeValidator(const void* /*value*/) const { return true; } @@ -276,8 +279,6 @@ class CommandLineFlag { flags_internal::FlagSettingMode set_mode, flags_internal::ValueSource source, std::string* error); - void StoreAtomic(size_t size); - void CheckDefaultValueParsingRoundtrip() const; // Constant configuration for a particular flag. @@ -300,11 +301,6 @@ class CommandLineFlag { void* cur_; // Lazily initialized pointer to current value int64_t counter_; // Mutation counter - // For some types, a copy of the current value is kept in an atomically - // accessible field. - static const int64_t kAtomicInit = 0xababababababababll; - std::atomic<int64_t> atomic_; - // Lazily initialized mutexes for this flag value. We cannot inline a // SpinLock or Mutex here because those have non-constexpr constructors and // so would prevent constant initialization of this type. |