When passing a lambda instead of a delegate to connect, it causes the app to segfault. Tested with DMD 2.085.1. Here's the most basic sample app that will reproduce the issue: ========== import std.signals; import std.stdio; class Test { mixin Signal!(int); void run () { emit (1); } } class Tester { void initialize () { Test test = new Test (); test.connect ((int a) => trigger (a)); } void trigger (int a) { writeln (a); } } void main () { Tester t = new Tester (); t.initialize (); } ==========
Lambda is a delegate in this context. I'm testing with v2.096.0 release canditate and latest Phobos. This bug does not trigger only with lambda, it triggers with any delegate that is not a class member function. The documentation does warn that the delegates must be class delegates, but it does not say that that they must be member function pointers. This needs to be clarified, or preferably it should be made to work with non-member-function delegates too.
@dukc created dlang/phobos pull request #8137 "Issue 19842 - only class member functions must be used be used as slots" mentioning this issue: - Issue 19842 - only class member functions must be used be used as slots https://github.com/dlang/phobos/pull/8137
@dukc created dlang/phobos pull request #8139 "Issue 19842 - only class member functions must be used be used as slots" mentioning this issue: - Issue 19842 - only class member functions must be used be used as slots https://github.com/dlang/phobos/pull/8139
@dukc created dlang/phobos pull request #8140 "Issue 19842 - only class member functions must be used be used as slots" mentioning this issue: - Issue 19842 - only class member functions must be used be used as slots https://github.com/dlang/phobos/pull/8140
dlang/phobos pull request #8140 "Issue 19842 - only class member functions must be used be used as slots" was merged into stable: - 235d3ba9bedf8cd7a9f697231b03f37bac9fd7d3 by Ate Eskola: Issue 19842 - only class member functions must be used be used as slots https://github.com/dlang/phobos/pull/8140
@MartinNowak created dlang/phobos pull request #8172 "merge stable" mentioning this issue: - Issue 19842 - only class member functions must be used be used as slots https://github.com/dlang/phobos/pull/8172
dlang/phobos pull request #8172 "merge stable" was merged into master: - 06983381cf824cfae7b8a77d29b0b2fa8fe13c6f by Ate Eskola: Issue 19842 - only class member functions must be used be used as slots https://github.com/dlang/phobos/pull/8172
The documetation is now clarified, but it still would be a nice plus for the slot to accept any delegate of the correct type. I changed the severity to enhancement.
THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/phobos/issues/9772 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB