[Lldb-commits] [lldb] [lldb][NFCI] Cleanup AppleObjCClassDescriptorV2::objc_class_t API (PR #200180)

[Raphael Isemann via lldb-commits]

================
@@ -725,12 +724,15 @@ uint64_t ClassDescriptorV2::GetInstanceSize() {
 static uint8_t IS_SWIFT_STABLE = 1U << 1;
 
 LanguageType ClassDescriptorV2::GetImplementationLanguage() const {
-  std::unique_ptr<objc_class_t> objc_class;
-  if (auto *process = m_runtime.GetProcess())
-    if (Read_objc_class(process, objc_class))
+  if (auto *process = m_runtime.GetProcess()) {
+    auto objc_class = objc_class_t::Read(process, m_objc_class_ptr);
----------------
Teemperor wrote:

I feel like it should be clearer from the code that this is an Expected, but as it's an improvement over the current code in any case I think this can be fixed in the future.

https://github.com/llvm/llvm-project/pull/200180