Skip to content

Conversation

@mrcsms
Copy link
Contributor

@mrcsms mrcsms commented Aug 19, 2021

... function CreateVarParam(PyDelphiWrapper : TPyDelphiWrapper; AObject: TObject) : PPyObject; overload;

@pyscripter
Copy link
Owner

Could you please describe a use case or even better a unit test that should pass?

@mrcsms
Copy link
Contributor Author

mrcsms commented Aug 19, 2021

it's not very common - for example, TcxGrid from DevExpress has an event defined as:
TcxGridGetPropertiesEvent = procedure(Sender: TcxCustomGridTableItem; ARecord: TcxCustomGridRecord; var AProperties: TcxCustomEditProperties) of object;
so event handler code use this case of CreateVarParam for passing AProperties to python side

@pyscripter
Copy link
Owner

I am happy to merge this, but is it unreasonable to ask to provide a unit test (in WrapDelphiTest.pas) before merging? This would serve as a documentation for users.

@mrcsms
Copy link
Contributor Author

mrcsms commented Aug 22, 2021

you are right, code should be covered by unit tests, I'll prepare sth

@mrcsms
Copy link
Contributor Author

mrcsms commented Sep 17, 2021

finally i added a test for this function - but to take a step forward, this test covers more general event handlers in p4d

@pyscripter pyscripter merged commit d45b048 into pyscripter:master Sep 17, 2021
@mrcsms mrcsms deleted the feature/CreateVarParam-with-object branch September 17, 2021 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants