On 6/11/19 6:25 AM, Martin Kletzander wrote:
> There are few more things that could be cleaned up related to the coding style
> and other things, like explicitly specifying the abi style after "extern"
> (i.e. `extern "C" fn` instead of `extern fn`), but since those are 
> configurable
> in rustfmt config, I'm not sure whether the config needs to be added or
> complying with the defaults should be the priority.  But this was just 
> something
> I stumbled upon when I saw some warnings during build, nothing more.
> 
> Martin Kletzander (2):
>   rust: Do not use deprecated integer types
>   rust: Remove libc dependency
> 
>  plugins/rust/Cargo.toml.in       |  1 -
>  plugins/rust/examples/ramdisk.rs | 15 ++++++-------
>  plugins/rust/src/lib.rs          | 37 +++++++++++++++-----------------
>  3 files changed, 24 insertions(+), 29 deletions(-)

I'm no Rust expert, but it doesn't break the build and doesn't look bad
to me. Pushed.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Libguestfs mailing list
Libguestfs@redhat.com
https://www.redhat.com/mailman/listinfo/libguestfs

Reply via email to