No bounds on structs

[jordens]

Unless absolutely necessary.

• minimq/src/mqtt_client.rs

  Line 52 in 2b8b2ee

  struct ClientContext<'a, Clock: embedded_time::Clock> {

• minimq/src/mqtt_client.rs

  Lines 263 to 268 in 2b8b2ee

  	pub struct MqttClient<
  	'buf,
  	TcpStack: TcpClientStack,
  	Clock: embedded_time::Clock,
  	Broker: crate::Broker,
  	> {

• minimq/src/mqtt_client.rs

  Lines 677 to 680 in 2b8b2ee

  	where
  	TcpStack: TcpClientStack,
  	Clock: embedded_time::Clock,
  	Broker: crate::Broker,

• minimq/src/config.rs

  Line 19 in 2b8b2ee

  pub(crate) struct Config<'a, Broker: crate::Broker> {

• minimq/src/config.rs

  Line 33 in 2b8b2ee

  pub struct ConfigBuilder<'a, Broker: crate::Broker> {

and others

ref https://stackoverflow.com/questions/49229332/should-trait-bounds-be-duplicated-in-struct-and-impl

[ryan-summers]

I think the only cleanup we can do is remove the broker bound because Instant has a bound on the construction parameter: https://docs.rs/embedded-time/latest/embedded_time/struct.Instant.html