[v2] Map more functions in stream module

[libvirt] [rust PATCH v2 0/5] Map more functions in stream module

Zixing Liu posted 5 patches 4 years, 3 months ago

Diff against v1 v3
Download series mbox

Failed in applying to current master (apply log)

There is a newer version of this series

src/domain.rs   |   2 +-
src/stream.rs   | 129 +++++++++++++++++++++++++++++++++++++++++-------
tests/stream.rs |  40 +++++++++++++++
3 files changed, 153 insertions(+), 18 deletions(-)
create mode 100644 tests/stream.rs

Expand all Fold all

[libvirt] [rust PATCH v2 0/5] Map more functions in stream module

Posted by Zixing Liu 4 years, 3 months ago

This set of patches will add more functions to the Rust bindings.
Newly mapped functions from C library: virStreamNew virStreamEventUpdateCallback virStreamEventRemoveCallback virStreamEventAddCallback.

virStreamEventAddCallback can accept normal fn functions or closures (can capture variables outside)

The changes are not very thoroughly tested since event module is not implemented at all so the virStreamEventAddCallback will always return "unsupported by the connection driver".

Also the changes have run though the rustfmt to ensure the format conforms to Rust officially recommended code style.

Version 2: Addressed comments

Zixing Liu (5):
  libvirt-rust: stream: add more functions in stream
  libvirt-rust: stream: add more functions in stream
  libvirt-rust: stream: automated lint
  libvirt-rust: use reference instead of moving
  libvirt-rust: stream: addressed comments

 src/domain.rs   |   2 +-
 src/stream.rs   | 129 +++++++++++++++++++++++++++++++++++++++++-------
 tests/stream.rs |  40 +++++++++++++++
 3 files changed, 153 insertions(+), 18 deletions(-)
 create mode 100644 tests/stream.rs

-- 
2.24.1


--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list