[PR #16] [MERGED] Instance handling refactoring (1 QML instance per .Net instance) #168

New issue

Closed

opened 2026-05-05 11:20:06 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 11:20:06 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/qmlnet/qmlnet/pull/16
Author: @devmil
Created: 7/19/2018
Status: ✅ Merged
Merged: 7/19/2018
Merged by: @pauldotknopf

Base: signals ← Head: features/InstanceCount

📝 Commits (6)

308af4f One NetValue for one NetInstance
6d4ac3e One NetInstance per .Net object (basically an attached property)
311c553 NetInstance: additional path for already released GCHandle references
0f98431 More robust ConditionalWeakTable usage
d216e93 Sandbox test for instance ref handling
00ceda3 A little bit more advanced instance testing in the Sandbox

📊 Changes

10 files changed (+197 additions, -19 deletions)

View changed files

📝 src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValue.cpp (+27 -8)
📝 src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValue.h (+10 -1)
📝 src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValueMetaObject.cpp (+1 -2)
📝 src/net/Qt.NetCore.Sandbox/Program.UI.cs (+50 -1)
📝 src/net/Qt.NetCore.Sandbox/main.qml (+84 -0)
📝 src/net/Qt.NetCore.Tests/Qml/NetVariantTests.cs (+1 -1)
📝 src/net/Qt.NetCore.Tests/Types/CallbacksTests.cs (+2 -2)
📝 src/net/Qt.NetCore.Tests/Types/NetInstanceTests.cs (+1 -1)
📝 src/net/Qt.NetCore/Internal/DefaultCallbacks.cs (+1 -1)
📝 src/net/Qt.NetCore/Types/NetInstance.cs (+20 -2)

📄 Description

No description provided

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/qmlnet/qmlnet/pull/16 **Author:** [@devmil](https://github.com/devmil) **Created:** 7/19/2018 **Status:** ✅ Merged **Merged:** 7/19/2018 **Merged by:** [@pauldotknopf](https://github.com/pauldotknopf) **Base:** `signals` ← **Head:** `features/InstanceCount` --- ### 📝 Commits (6) - [`308af4f`](https://github.com/qmlnet/qmlnet/commit/308af4fbafd560e3b79f0cfb964da70c20c49d81) One NetValue for one NetInstance - [`6d4ac3e`](https://github.com/qmlnet/qmlnet/commit/6d4ac3eabeda46d72b8b79fae001e81791683880) One NetInstance per .Net object (basically an attached property) - [`311c553`](https://github.com/qmlnet/qmlnet/commit/311c55364627b63209189cfd4be7a78d257d259a) NetInstance: additional path for already released GCHandle references - [`0f98431`](https://github.com/qmlnet/qmlnet/commit/0f984313d626838898f07033f8454c6444426415) More robust ConditionalWeakTable usage - [`d216e93`](https://github.com/qmlnet/qmlnet/commit/d216e9309e9b7856dfd3163bf2069ca41c355ea2) Sandbox test for instance ref handling - [`00ceda3`](https://github.com/qmlnet/qmlnet/commit/00ceda30be09ecc7ed178b874c9521048ddc7d4b) A little bit more advanced instance testing in the Sandbox ### 📊 Changes **10 files changed** (+197 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValue.cpp` (+27 -8) 📝 `src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValue.h` (+10 -1) 📝 `src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValueMetaObject.cpp` (+1 -2) 📝 `src/net/Qt.NetCore.Sandbox/Program.UI.cs` (+50 -1) 📝 `src/net/Qt.NetCore.Sandbox/main.qml` (+84 -0) 📝 `src/net/Qt.NetCore.Tests/Qml/NetVariantTests.cs` (+1 -1) 📝 `src/net/Qt.NetCore.Tests/Types/CallbacksTests.cs` (+2 -2) 📝 `src/net/Qt.NetCore.Tests/Types/NetInstanceTests.cs` (+1 -1) 📝 `src/net/Qt.NetCore/Internal/DefaultCallbacks.cs` (+1 -1) 📝 `src/net/Qt.NetCore/Types/NetInstance.cs` (+20 -2) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>