don't complain about button pin = -1

softhack007 · softhack007 · commit 917cd96a3d15 · 2022-12-10T21:03:58.000+01:00
diff --git a/wled00/set.cpp b/wled00/set.cpp
@@ -174,7 +174,7 @@ void handleSettingsSet(AsyncWebServerRequest *request, byte subPage)
         if (((buttonType[i] == BTN_TYPE_ANALOG) || (buttonType[i] == BTN_TYPE_ANALOG_INVERTED)) && (digitalPinToAnalogChannel(btnPin[i]) < 0)) 
         {
           // not an ADC analog pin
-          DEBUG_PRINTF("PIN ALLOC error: GPIO%d for analog button #%d is not an analog pin!\n", btnPin[i], i);
+          if (btnPin[i] >= 0) DEBUG_PRINTF("PIN ALLOC error: GPIO%d for analog button #%d is not an analog pin!\n", btnPin[i], i);
           btnPin[i] = -1;
           pinManager.deallocatePin(hw_btn_pin,PinOwner::Button);
         } 

Original file line number	Diff line number	Diff line change
`@@ -174,7 +174,7 @@ void handleSettingsSet(AsyncWebServerRequest *request, byte subPage)`
`174`	`174`	`if (((buttonType[i] == BTN_TYPE_ANALOG) \|\| (buttonType[i] == BTN_TYPE_ANALOG_INVERTED)) && (digitalPinToAnalogChannel(btnPin[i]) < 0))`
`175`	`175`	`{`
`176`	`176`	`// not an ADC analog pin`
`177`		`- DEBUG_PRINTF("PIN ALLOC error: GPIO%d for analog button #%d is not an analog pin!\n", btnPin[i], i);`
	`177`	`+ if (btnPin[i] >= 0) DEBUG_PRINTF("PIN ALLOC error: GPIO%d for analog button #%d is not an analog pin!\n", btnPin[i], i);`
`178`	`178`	`btnPin[i] = -1;`
`179`	`179`	`pinManager.deallocatePin(hw_btn_pin,PinOwner::Button);`
`180`	`180`	`}`