Skip to content

Conversation

@xyjixyjixyji
Copy link
Contributor

@xyjixyjixyji xyjixyjixyji commented Apr 9, 2023

  • Add the freeing of the operator
  • Modify the test to free the operator after the test

* Add opendal_operator_free
* Add opendal_operator_free to test

Signed-off-by: Ji-Xinyou <jerryji0414@outlook.com>
@xyjixyjixyji xyjixyjixyji changed the title feat(bindings/c): free, limit operation and tests feat(bindings/c): add support for free heap-allocated operator Apr 9, 2023
@xyjixyjixyji xyjixyjixyji requested a review from Xuanwo April 9, 2023 13:32
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@Xuanwo Xuanwo merged commit 391a56c into apache:main Apr 9, 2023
@xyjixyjixyji xyjixyjixyji deleted the c-more branch April 9, 2023 16:38
wcy-fdu pushed a commit to wcy-fdu/opendal that referenced this pull request Apr 12, 2023
…e#1890)

feat(bindings/c): add free op

* Add opendal_operator_free
* Add opendal_operator_free to test

Signed-off-by: Ji-Xinyou <jerryji0414@outlook.com>
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.

3 participants