Repository navigation
New system requires class as part of the type #4
Description
I am not sure if this was intentional but the new implementation requires that the type of the class be part of the template type for member functions. This was not the case with the previous implementation. In other words, the type of a delegate now restricts it for use on a certain type.
I would assume that this is not part of the plan because if this is allowable then the concept of this delegate system does not make it any more useful than standard C++ member function pointers. Or am I missing something?
Activity
[-]New system requires Class as part of the type[/-][+]New system requires class as part of the type[/+]timoalho commentedon Jun 11, 2016
I have to agree with this, the fact that the type of the caller appears in the type of the delegate seems to defeat the whole purpose of the delegate...
If I say, for example
then
does not compile.
jcelerier commentedon Jul 22, 2016
+1, bitten with this.
You should at least change the examples on the front page because they give hope that the delegate is able to do this.
jcelerier commentedon Nov 22, 2016
@marcmo Wouldn't C++17's template argument deduction help overcome this ?
marcmo commentedon Nov 23, 2016
you are of course right guys...I overlooked this in my enthusiasm to get rid of some template syntax by using macros...
will probably switch back to using the more verbose but working syntax. @jcelerier C++17 might help but then again this library was meant to be used in scenarios where you cannot expect more then C++98 => highly portable but still really good performance.
koemeet commentedon Dec 31, 2017
just noticed this as well, any plans on maybe providing a working branch or tag a certain commit in which it was working as described in this issue?
marcmo commentedon Jan 2, 2018
Hi @koemeet, I will provide an updated version soon. totally forgot about it.
hparracho commentedon Mar 30, 2018
Hi @marcmo! Was this issue dropped or do you still have plans to update the class?