search

ruabmbua / hidapi-rs

Rust bindings for the hidapi C library
MIT License
172 stars 81 forks source link

Fixed issue #5 & some minor changes #9

Closed ruabmbua closed 7 years ago

ruabmbua commented 7 years ago

This fixes the bug in issue #5 without breaking API compatibility.

Also some minor changes here and there like updating doc & fixing doc.rs link.

@r8d8 could you please test it with 0 devices, I struggle at removing all hid devices (is a builtin kernel module, and my laptop has some integrated hid peripherals).

Edit: Although I am almost sure it will work.

r8d8 commented 7 years ago

Yeap, offcourse

r8d8 commented 7 years ago

Side question: I was thinking that all laptops should have at least 1 HID device(keyboard), or it is implemented in other way?

ruabmbua commented 7 years ago

Not really (mine uses HID for the keyboard lights, and PS/2 for keyboard)

I am almost sure there are some without any HID devices.

r8d8 commented 7 years ago

@ruabmbua it works, no Seq Faults

ruabmbua commented 7 years ago

Ping @Osspial , could you please review & publish.

Osspial commented 7 years ago

Looks good, publishing now.