Closed martin-ejdestig closed 5 years ago
Thought about submitting a PR removing connect() entirely as well but opened #54 instead.
Should perhaps mention that the latest version of the code that I added has not been test compiled. Will do that tomorrow if reviewer does not beat me to it. :)
Tested and it works after fixing name of MessageHelper. Also added a commit that replaces "virtual" with "override".
Was confusing to have connect() with a "deprecated" comment in the generated stub header and not being sure what to do instead.