Reorder WireBusDevice create functions to form a natural progression from few parameters to many.

Author: aentinger

src/BusDeviceCreator.cpp

@@ -43,19 +43,19 @@ SpiBusDevice BusDeviceCreator::create(arduino::SPIClass & spi, SPISettings const
   return SpiBusDevice(SpiBusDeviceConfig{spi, spi_settings, cs_pin, fill_symbol});
 }
 
-WireBusDevice BusDeviceCreator::create(arduino::HardwareI2C & wire, byte const slave_addr, bool const restart, bool const stop)
+WireBusDevice BusDeviceCreator::create(arduino::HardwareI2C & wire, byte const slave_addr)
 {
-  return WireBusDevice(WireBusDeviceConfig{wire, slave_addr, restart, stop});
+  return create(wire, slave_addr, true, true);
 }
 
 WireBusDevice BusDeviceCreator::create(arduino::HardwareI2C & wire, byte const slave_addr, bool const restart)
 {
   return create(wire, slave_addr, restart, true);
 }
 
-WireBusDevice BusDeviceCreator::create(arduino::HardwareI2C & wire, byte const slave_addr)
+WireBusDevice BusDeviceCreator::create(arduino::HardwareI2C & wire, byte const slave_addr, bool const restart, bool const stop)
 {
-  return create(wire, slave_addr, true, true);
+  return WireBusDevice(WireBusDeviceConfig{wire, slave_addr, restart, stop});
 }
 
 /**************************************************************************************

src/BusDeviceCreator.h

@@ -40,9 +40,9 @@ class BusDeviceCreator
   SpiBusDevice create(arduino::SPIClass & spi, SPISettings const & spi_settings, SpiBusDeviceConfig::SpiSelectFunc spi_select, SpiBusDeviceConfig::SpiDeselectFunc spi_deselect, byte const fill_symbol = 0xFF);
   SpiBusDevice create(arduino::SPIClass & spi, SPISettings const & spi_settings, int const cs_pin, byte const fill_symbol = 0xFF);
 
-  WireBusDevice create(arduino::HardwareI2C & wire, byte const slave_addr, bool const restart, bool const stop);
-  WireBusDevice create(arduino::HardwareI2C & wire, byte const slave_addr, bool const restart);
   WireBusDevice create(arduino::HardwareI2C & wire, byte const slave_addr);
+  WireBusDevice create(arduino::HardwareI2C & wire, byte const slave_addr, bool const restart);
+  WireBusDevice create(arduino::HardwareI2C & wire, byte const slave_addr, bool const restart, bool const stop);
 
 };