Here I want to discuss and implement changes to make invoke_method (and maybe things around it) better.
At first, I've changed invoke_method's return value to Option<QVariant> - which is close to original QMetaObject API and makes it possible to check wether error occurred (this was discussed earlier).
@ogoffart, can you give me advice -- is this change OK as it done now? What should be done next?
IriaSomobu
commented
9 months ago
This PR is a follow-up of https://github.com/woboq/qmetaobject-rs/pull/302. This PR introduces breaking changes and not to ready to be merged (yet).
Here I want to discuss and implement changes to make
invoke_method(and maybe things around it) better.At first, I've changed
invoke_method's return value toOption<QVariant>- which is close to original QMetaObject API and makes it possible to check wether error occurred (this was discussed earlier).@ogoffart, can you give me advice -- is this change OK as it done now? What should be done next?