diff options
| author | Adrian Prantl <aprantl@apple.com> | 2021-02-11 12:57:04 -0800 |
|---|---|---|
| committer | Adrian Prantl <aprantl@apple.com> | 2021-02-12 16:12:31 -0800 |
| commit | 057efa9916cdc354ef4653bcd128a578cc43125e (patch) | |
| tree | 2b71ac656745bb4ed134207cd1c63e914c543a52 /lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp | |
| parent | 532d4bf025442cb1357789bf6d9941bb9c5219c6 (diff) | |
Make the error condition in Value::ValueType explicit (NFC)
The comment for ValueType claims that all values <1 are errors, but
not all switch statements take this into account. This patch
introduces an explicit Error case and deletes all default: cases, so
we get warned about incomplete switch coverage.
https://reviews.llvm.org/D96537
Diffstat (limited to 'lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp')
| -rw-r--r-- | lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp b/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp index 3628b0a2ce5e..09ba83a4e94b 100644 --- a/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp +++ b/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp @@ -659,7 +659,7 @@ PlatformPOSIX::MakeLoadImageUtilityFunction(ExecutionContext &exe_ctx, // We are passing four arguments, the basename, the list of places to look, // a buffer big enough for all the path + name combos, and // a pointer to the storage we've made for the result: - value.SetValueType(Value::eValueTypeScalar); + value.SetValueType(Value::ValueType::Scalar); value.SetCompilerType(clang_void_pointer_type); arguments.PushValue(value); value.SetCompilerType(clang_char_pointer_type); |