cue-sdk icon indicating copy to clipboard operation
cue-sdk copied to clipboard

CorsairGetDevices() lacks documentation and provides a useless error.

Open jrlanglois opened this issue 5 months ago • 1 comments

This API is particularly confusing, suffice to say. Apparently if you provide a very large number of devices to request, you get a vague error: CE_InvalidArguments.

This is not ideal for a few reasons:

  1. I have no idea if there's an upper limit as there aren't any outlined.
    • A manual test shows the upper boundary to be 64.
  2. There's no API to call to simply give me all of the devices with the given filter.
    • It could simply provide an "out" parameter with the number.
  3. The error doesn't tell me which argument is invalid, nor does it tell me why.

Can you please provide better in-code docs at the very least.

jrlanglois avatar Sep 20 '24 18:09 jrlanglois