VideoStreaming status processing

[alexey-pelykh]

No description provided.

[ktossell]

It makes sense to strip off the first 5 bytes when delivering this update, but 1) this breaks any existing code that might be relying on the update's content and 2) the originator ID is lost.

I'd be open to adding an originator parameter to the callback type, maybe between status_class and event, and also making the 5-byte-offset change. That way, anything that uses the callback will break, but at least the breakage will happen at compile time.

[ktossell]

Or this may be a good time to do:

struct uvc_status_update {
  enum uvc_status_class status_class;
  int originator;
  int event;
  int selector;
  enum uvc_status_attribute status_attribute;
  void *data;
  size_t data_len;
};

typedef void(uvc_status_callback_t)(struct uvc_status_update, void* user_ptr);

or something like that.

[alexey-pelykh]

I'd suggest to add these interface-breaking changes in other pull-request

[alexey-pelykh]

Summary:

• First 5 bytes strip-off: already done, and it's not a breaking change since it was before
• Originator & arguments-as-structure: Right now behaviour is the same, and this change would be better to do in other pull-request

[ktossell]

First 5 bytes strip-off: already done,

So it was! Will merge.