HackStudio: Store correct address in ClassViewModel's ModelIndices

When 359d6e7b0b happened, the return value of `children[row]` went from being `ClassViewNode&` to `NonnullOwnPtr<ClassViewNode>&`, so we were putting the wrong address into the ModelIndex's data.
Author: https://github.com/AtkinsSJ Commit: https://github.com/SerenityOS/serenity/commit/24d5bf8173 Pull-request: https://github.com/SerenityOS/serenity/pull/17773 Issue: https://github.com/SerenityOS/serenity/issues/17755 Reviewed-by: https://github.com/nico ✅
2023-03-08 20:51:25 +00:00 · 2023-03-08 20:51:25 +00:00 · 24d5bf8173 · 2024-07-16 23:10:13 +09:00
commit 24d5bf8173
parent 1393ed2000
1 changed files with 2 additions and 2 deletions
--- a/Userland/DevTools/HackStudio/ClassViewWidget.cpp
+++ b/Userland/DevTools/HackStudio/ClassViewWidget.cpp
@ -91,9 +91,9 @@ GUI::ModelIndex ClassViewModel::parent_index(const GUI::ModelIndex& index) const
 GUI::ModelIndex ClassViewModel::index(int row, int column, const GUI::ModelIndex& parent_index) const
 {
    if (!parent_index.is_valid())
-        return create_index(row, column, &m_root_scope[row]);
+        return create_index(row, column, m_root_scope[row].ptr());
    auto* parent = static_cast<ClassViewNode const*>(parent_index.internal_data());
-    auto* child = &parent->children[row];
+    auto* child = parent->children[row].ptr();
    return create_index(row, column, child);
 }