Inverted the 'Device Already Open' check so it will work as designed. (#2095)

gjnijenhuis · gjnijenhuis-sensata · web-flow · commit bd3eb2d6833e · 2023-07-08T12:57:12.000+02:00
As per issue #2094 Co-authored-by: Gido Nijenhuis <g-nijenhuis@sensata.com>
diff --git a/src/devices/Ft232H/Ft232HI2cBus.cs b/src/devices/Ft232H/Ft232HI2cBus.cs
@@ -37,7 +37,7 @@ public Ft232HI2cBus(Ftx232HDevice deviceInformation)
         /// <inheritdoc/>
         public override I2cDevice CreateDevice(int deviceAddress)
         {
-            if (!_usedAddresses.ContainsKey(deviceAddress))
+            if (_usedAddresses.ContainsKey(deviceAddress))
             {
                 throw new ArgumentException($"Device with address 0x{deviceAddress,0X2} is already open.", nameof(deviceAddress));
             }

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ public Ft232HI2cBus(Ftx232HDevice deviceInformation)`
`37`	`37`	`/// <inheritdoc/>`
`38`	`38`	`public override I2cDevice CreateDevice(int deviceAddress)`
`39`	`39`	`{`
`40`		`- if (!_usedAddresses.ContainsKey(deviceAddress))`
	`40`	`+ if (_usedAddresses.ContainsKey(deviceAddress))`
`41`	`41`	`{`
`42`	`42`	`throw new ArgumentException($"Device with address 0x{deviceAddress,0X2} is already open.", nameof(deviceAddress));`
`43`	`43`	`}`