[PR #17] [MERGED] Added support for declaring and raising signals. #170

New issue

Closed

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

gitea-mirror commented

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

Owner

📋 Pull Request Information

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

Base: master ← Head: signals

📝 Commits (10+)

8f3d3fd Added ability to add signals to type info.
3bbf573 Added the signal info wrapping type.
cf8abed Using the new signal type on the type info.
eda3d26 Added support for discovering signals on the type info.
6fa1472 Added support for activating the signal from qml.
df55652 This is the failure scenario.
5cb7220 One NetValue for one NetInstance
3119c41 One NetInstance per .Net object (basically an attached property)
7121405 NetInstance: additional path for already released GCHandle references
db37763 More robust ConditionalWeakTable usage

📊 Changes

28 files changed (+1030 additions, -155 deletions)

View changed files

📝 src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValue.cpp (+90 -7)
📝 src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValue.h (+12 -1)
📝 src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValueMetaObject.cpp (+38 -3)
📝 src/native/QtNetCoreQml/QtNetCoreQml/types/NetInstance.cpp (+36 -4)
📝 src/native/QtNetCoreQml/QtNetCoreQml/types/NetInstance.h (+2 -0)
➕ src/native/QtNetCoreQml/QtNetCoreQml/types/NetSignalInfo.cpp (+55 -0)
➕ src/native/QtNetCoreQml/QtNetCoreQml/types/NetSignalInfo.h (+24 -0)
📝 src/native/QtNetCoreQml/QtNetCoreQml/types/NetTypeInfo.cpp (+32 -0)
📝 src/native/QtNetCoreQml/QtNetCoreQml/types/NetTypeInfo.h (+6 -0)
📝 src/native/QtNetCoreQml/QtNetCoreQml/types/types.pri (+4 -2)
📝 src/net/Qt.NetCore.Sandbox/Program.UI.cs (+56 -12)
📝 src/net/Qt.NetCore.Sandbox/Qt.NetCore.Sandbox.csproj (+3 -0)
📝 src/net/Qt.NetCore.Sandbox/main.qml (+126 -3)
📝 src/net/Qt.NetCore.Tests/Qml/NetVariantTests.cs (+1 -1)
➕ src/net/Qt.NetCore.Tests/Qml/SignalTests.cs (+182 -0)
📝 src/net/Qt.NetCore.Tests/Types/CallbacksTests.cs (+2 -2)
📝 src/net/Qt.NetCore.Tests/Types/NetInstanceTests.cs (+1 -10)
➕ src/net/Qt.NetCore.Tests/Types/NetSignalInfoTests.cs (+24 -0)
📝 src/net/Qt.NetCore.Tests/Types/NetTypeInfoTests.cs (+15 -0)
📝 src/net/Qt.NetCore.Tests/Types/NetTypeManagerTests.cs (+19 -0)

...and 8 more files

📄 Description

Closes #7

_{🔄 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/17 **Author:** [@pauldotknopf](https://github.com/pauldotknopf) **Created:** 7/20/2018 **Status:** ✅ Merged **Merged:** 7/20/2018 **Merged by:** [@pauldotknopf](https://github.com/pauldotknopf) **Base:** `master` ← **Head:** `signals` --- ### 📝 Commits (10+) - [`8f3d3fd`](https://github.com/qmlnet/qmlnet/commit/8f3d3fd6b2ebaf3466e78ea0a85fdea5ad4e8430) Added ability to add signals to type info. - [`3bbf573`](https://github.com/qmlnet/qmlnet/commit/3bbf573002e3acf9aad01f13e920c15c48a5065f) Added the signal info wrapping type. - [`cf8abed`](https://github.com/qmlnet/qmlnet/commit/cf8abed3448aa83f9991242bb8b5c82ee96ac0fa) Using the new signal type on the type info. - [`eda3d26`](https://github.com/qmlnet/qmlnet/commit/eda3d2603136a98e11544e0da50c547bad4c7d06) Added support for discovering signals on the type info. - [`6fa1472`](https://github.com/qmlnet/qmlnet/commit/6fa14724c88abd35c0308b5340095d524f3a7219) Added support for activating the signal from qml. - [`df55652`](https://github.com/qmlnet/qmlnet/commit/df55652c189bccd24da4936aefbf4e8bf3f9e6e6) This is the failure scenario. - [`5cb7220`](https://github.com/qmlnet/qmlnet/commit/5cb7220cdc97d158efd24c866cdc93110379e9e5) One NetValue for one NetInstance - [`3119c41`](https://github.com/qmlnet/qmlnet/commit/3119c413dbad86528a3e32a124e260b9806a6a45) One NetInstance per .Net object (basically an attached property) - [`7121405`](https://github.com/qmlnet/qmlnet/commit/7121405ee8bc130da0da8426728c064f0678e80f) NetInstance: additional path for already released GCHandle references - [`db37763`](https://github.com/qmlnet/qmlnet/commit/db377639a466fc208f4ea9fd0a7d0188311d0625) More robust ConditionalWeakTable usage ### 📊 Changes **28 files changed** (+1030 additions, -155 deletions) <details> <summary>View changed files</summary> 📝 `src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValue.cpp` (+90 -7) 📝 `src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValue.h` (+12 -1) 📝 `src/native/QtNetCoreQml/QtNetCoreQml/qml/NetValueMetaObject.cpp` (+38 -3) 📝 `src/native/QtNetCoreQml/QtNetCoreQml/types/NetInstance.cpp` (+36 -4) 📝 `src/native/QtNetCoreQml/QtNetCoreQml/types/NetInstance.h` (+2 -0) ➕ `src/native/QtNetCoreQml/QtNetCoreQml/types/NetSignalInfo.cpp` (+55 -0) ➕ `src/native/QtNetCoreQml/QtNetCoreQml/types/NetSignalInfo.h` (+24 -0) 📝 `src/native/QtNetCoreQml/QtNetCoreQml/types/NetTypeInfo.cpp` (+32 -0) 📝 `src/native/QtNetCoreQml/QtNetCoreQml/types/NetTypeInfo.h` (+6 -0) 📝 `src/native/QtNetCoreQml/QtNetCoreQml/types/types.pri` (+4 -2) 📝 `src/net/Qt.NetCore.Sandbox/Program.UI.cs` (+56 -12) 📝 `src/net/Qt.NetCore.Sandbox/Qt.NetCore.Sandbox.csproj` (+3 -0) 📝 `src/net/Qt.NetCore.Sandbox/main.qml` (+126 -3) 📝 `src/net/Qt.NetCore.Tests/Qml/NetVariantTests.cs` (+1 -1) ➕ `src/net/Qt.NetCore.Tests/Qml/SignalTests.cs` (+182 -0) 📝 `src/net/Qt.NetCore.Tests/Types/CallbacksTests.cs` (+2 -2) 📝 `src/net/Qt.NetCore.Tests/Types/NetInstanceTests.cs` (+1 -10) ➕ `src/net/Qt.NetCore.Tests/Types/NetSignalInfoTests.cs` (+24 -0) 📝 `src/net/Qt.NetCore.Tests/Types/NetTypeInfoTests.cs` (+15 -0) 📝 `src/net/Qt.NetCore.Tests/Types/NetTypeManagerTests.cs` (+19 -0) _...and 8 more files_ </details> ### 📄 Description Closes #7 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>