Use 0 instead of 255 to disable buttons in ini file

.gitignore

@@ -41,3 +41,7 @@ MiniDexed*
 sdcard
 *.zip
 *.img
+
+# Editor related files
+*.swp
+*.swo

src/config.cpp

@@ -81,11 +81,11 @@ void CConfig::Load (void)
 	m_nLCDPinData7 = m_Properties.GetNumber ("LCDPinData7", 25);
 	m_nLCDI2CAddress = m_Properties.GetNumber ("LCDI2CAddress", 0);
 
-	m_nButtonPinPrev = m_Properties.GetNumber ("ButtonPinPrev", NOPIN);
-	m_nButtonPinNext = m_Properties.GetNumber ("ButtonPinNext", NOPIN);
-	m_nButtonPinBack = m_Properties.GetNumber ("ButtonPinBack", NOPIN);
-	m_nButtonPinSelect = m_Properties.GetNumber ("ButtonPinSelect", NOPIN);
-	m_nButtonPinHome = m_Properties.GetNumber ("ButtonPinHome", NOPIN);
+	m_nButtonPinPrev = m_Properties.GetNumber ("ButtonPinPrev", 0);
+	m_nButtonPinNext = m_Properties.GetNumber ("ButtonPinNext", 0);
+	m_nButtonPinBack = m_Properties.GetNumber ("ButtonPinBack", 0);
+	m_nButtonPinSelect = m_Properties.GetNumber ("ButtonPinSelect", 0);
+	m_nButtonPinHome = m_Properties.GetNumber ("ButtonPinHome", 0);
 
 	m_bEncoderEnabled = m_Properties.GetNumber ("EncoderEnabled", 0) != 0;
 	m_nEncoderPinClock = m_Properties.GetNumber ("EncoderPinClock", 10);

src/config.h

@@ -28,9 +28,6 @@
 #include <circle/sysconfig.h>
 #include <string>
 
-// This has to match the value mentioned in minidexed.ini
-#define NOPIN 255
-
 class CConfig		// Configuration for MiniDexed
 {
 public:

src/minidexed.ini

@@ -32,7 +32,7 @@ LCDI2CAddress=0x00
 #    5 Buttons based on an "arrow keypad" with a "home"
 #    2 Buttons "back" and "select" for use in addition to a rotary encoder
 #
-#  Although actually any buttons set to 255 will be ignored
+#  Any buttons set to 0 will be ignored
 #
 ButtonPinPrev=16
 ButtonPinNext=20

src/uibuttons.cpp

@@ -45,7 +45,7 @@ boolean CUIButton::Initialize (void)
 {
 	assert (!m_pPin);
 
-	if (m_nPin != NOPIN)
+	if (m_nPin != 0)
 	{
 		m_pPin = new CGPIOPin (m_nPin, GPIOModeInputPullUp);
 	}

src/uibuttons.h

@@ -59,7 +59,7 @@ public:
 	typedef void TBtnEventHandler (TBtnEvent Event, void *pParam);
 
 public:
-	CUIButtons (unsigned nPrevPin = NOPIN, unsigned nNextPin = NOPIN, unsigned nBackPin = NOPIN, unsigned nSelectPin = NOPIN, unsigned nHomePin = NOPIN);
+	CUIButtons (unsigned nPrevPin = 0, unsigned nNextPin = 0, unsigned nBackPin = 0, unsigned nSelectPin = 0, unsigned nHomePin = 0);
 	~CUIButtons (void);
 	
 	boolean Initialize (void);