Derive Debug for all structs

[h7x4]

Let all structs derive debug for ease of debugging.

The only thing that I feel is a little questionable about this is making UsbInterfaceHandler and UsbDeviceHandler into a supertraits of Debug. Although not semantically correct, it is meant to force UsbDevice.device_handler: Option<Arc<Mutex<Box<dyn UsbDeviceHandler + Send>>>> to be debuggable, so UsbDevice can be so as well. Would it be better to maintain a manual implementation here?

[jiegec]

I don't know what's the best practice, but it might break existing code not impl-ing Debug?

[h7x4]

... it might break existing code not impl-ing Debug?

That's a good call. Do you think it would be better to create a manual implementation? I guess something like derivative also is an option, at the cost of introducing a new dependency.

[jamesadevine]

Could this be a package configuration option instead?

[h7x4]

That works for me. I've never heard of debug being a used as a configuration option though. Is the intention to include the extra dependency only for those that need it?

[jamesadevine]

I don't think it is a big deal to derive the debug struct personally.