Amr Bekhit

Some of the config module functions (e.g. conf_get_value) don't work when the string is a const. Because these functions internally use strtok_r, which modifies the original string, if the string...

The `WatchDeviceChanges` function creates a goroutine that watches for any change notifications coming from the underlying Device. The result of `b.Parse` is sent to the channel `ch` using a blocking...

The `WatchDeviceChanges` function creates a goroutine that watches for any change notifications coming from the underlying Device. The result of `b.Parse` is sent to the channel `ch` using a blocking...

Sending `nil` to `b.propchanged` when the context is done does not seem to serve any purpose. The `bluez.WatchProperties` function that creates `propchanged` does not read from it. The channel is...

This allows the module to be compiled using Go 1.14, as specified in the go.mod file. `reflect.Ptr` was renamed to `reflect.Pointer` in Go 1.18 (see https://tip.golang.org/doc/go1.18). The code uses `reflect.Pointer`...

When running `ykinfo -s -q` I noticed that the USB HID Keyboard interface is removed and re-added by the kernel. This does not happen when the equivalent `ykman list` command...