Modified MODSERIAL to compile with -Wall -Wextra -pedantic -Werror

lib/MODSERIAL/FLUSH.cpp

@@ -32,6 +32,7 @@ MODSERIAL::flushBuffer(IrqType type)
     switch(type) {
         case TxIrq: _IER &= ~(1UL << 1); break;
         case RxIrq: _IER &= ~(1UL << 0); break;
+        default: break;
     }
     buffer_in[type]       = 0;
     buffer_out[type]      = 0;
@@ -40,8 +41,9 @@ MODSERIAL::flushBuffer(IrqType type)
     switch(type) {
         case TxIrq: _FCR = MODSERIAL_FIFO_TX_RESET; break;
         case RxIrq: _FCR = MODSERIAL_FIFO_RX_RESET; break;
+        default: break;
     }
     _IER = ier;
 }
 
-}; // namespace AjK ends
+} // namespace AjK ends

lib/MODSERIAL/GETC.cpp

@@ -62,4 +62,4 @@ MODSERIAL::__getc(bool block)
     return c;
 }
 
-}; // namespace AjK ends
+} // namespace AjK ends

lib/MODSERIAL/INIT.cpp

@@ -27,7 +27,7 @@
 namespace AjK {
 
 void
-MODSERIAL::init( int txSize, int rxSize, PinName rx )
+MODSERIAL::init( int txSize, int rxSize )
 {
     disableIrq();
     
@@ -78,4 +78,4 @@ MODSERIAL::init( int txSize, int rxSize, PinName rx )
     enableIrq();
 }
 
-}; // namespace AjK ends
+} // namespace AjK ends

lib/MODSERIAL/ISR_RX.cpp

@@ -56,5 +56,5 @@ MODSERIAL::isr_rx(void)
     }    
 }
 
-}; // namespace AjK ends
+} // namespace AjK ends
 

lib/MODSERIAL/ISR_TX.cpp

@@ -49,6 +49,7 @@ MODSERIAL::isr_tx(bool doCallback)
     }        
 }
 
-}; // namespace AjK ends
+} // namespace AjK ends
+
 
         

lib/MODSERIAL/MODSERIAL.cpp

@@ -33,17 +33,17 @@ namespace AjK {
 
 MODSERIAL::MODSERIAL( PinName tx, PinName rx, const char* name ) : Serial( tx, rx, name )
 {
-    init( MODSERIAL_DEFAULT_TX_BUFFER_SIZE, MODSERIAL_DEFAULT_RX_BUFFER_SIZE, rx );
+    init( MODSERIAL_DEFAULT_TX_BUFFER_SIZE, MODSERIAL_DEFAULT_RX_BUFFER_SIZE );
 }
 
 MODSERIAL::MODSERIAL( PinName tx, PinName rx, int bufferSize, const char* name ) : Serial( tx, rx, name )
 {
-    init( bufferSize, bufferSize, rx );
+    init( bufferSize, bufferSize );
 }
 
 MODSERIAL::MODSERIAL( PinName tx, PinName rx, int txSize, int rxSize, const char* name ) : Serial( tx, rx, name )
 {
-    init( txSize, rxSize, rx );
+    init( txSize, rxSize );
 }
 
 MODSERIAL::~MODSERIAL()
@@ -77,11 +77,11 @@ MODSERIAL::rxBufferEmpty( void )
     return MODSERIAL_RX_BUFFER_EMPTY; 
 }
 
-bool 
-MODSERIAL::txIsBusy( void ) 
-{ 
-    return ( _LSR & ( 3UL << 5 ) == 0 ) ? true : false; 
-} 
+bool
+MODSERIAL::txIsBusy( void )
+{
+    return ( ( _LSR & ( 3UL << 5 ) ) == 0 ) ? true : false;
+}
 
 void
 MODSERIAL::disableIrq( void )
@@ -134,4 +134,4 @@ MODSERIAL::rxDiscardLastChar( void )
 }
 
 
-}; // namespace AjK ends
+} // namespace AjK ends

lib/MODSERIAL/MODSERIAL.h

@@ -41,7 +41,12 @@
 #define MODSERIAL_DEFAULT_TX_BUFFER_SIZE    256
 #endif
 
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wunused-parameter"
+#pragma GCC diagnostic ignored "-pedantic"
 #include "mbed.h"
+#pragma GCC diagnostic pop
+
 #include "serial_api.h"
 
 namespace AjK {
@@ -436,8 +441,8 @@ public:
      * @see attach
      * @ingroup API
      */
-    void connect(void (*fptr)(MODSERIAL_IRQ_INFO *), IrqType type = RxIrq) { _isr[RxIrq].attach(fptr); }
-    
+    void connect(void (*fptr)(MODSERIAL_IRQ_INFO *), IrqType type = RxIrq) { _isr[type].attach(fptr); }
+
     /**
      * @see attach
      * @ingroup API
@@ -517,8 +522,8 @@ public:
      * @ingroup API
      * @return The length iof the TX buffer in bytes
      */
-    int txBufferGetSize(int size) { return buffer_size[TxIrq]; } 
-    
+    int txBufferGetSize() { return buffer_size[TxIrq]; }
+
     /**
      * Function: rxBufferGetSize
      *  
@@ -527,8 +532,8 @@ public:
      * @ingroup API
      * @return The length iof the RX buffer in bytes
      */
-    int rxBufferGetSize(int size) { return buffer_size[RxIrq]; } 
-    
+    int rxBufferGetSize() { return buffer_size[RxIrq]; }
+
     /**
      * Function: txBufferFull
      *  
@@ -932,7 +937,7 @@ private:
      * Initialize the MODSERIAL object
      * @ingroup INTERNALS
      */
-    void init(int txSize, int rxSize, PinName rx);
+    void init(int txSize, int rxSize);
     
     /** 
      * Function: flushBuffer
@@ -1082,7 +1087,7 @@ protected:
 
 };
 
-}; // namespace AjK ends
+} // namespace AjK ends
 
 using namespace AjK;
 

lib/MODSERIAL/MODSERIAL_IRQ_INFO.cpp

@@ -35,4 +35,4 @@ MODSERIAL_IRQ_INFO::rxDiscardLastChar(void)
     return serial->rxDiscardLastChar(); 
 }
 
-}; // namespace AjK ends
+} // namespace AjK ends

lib/MODSERIAL/PUTC.cpp

@@ -78,4 +78,4 @@ MODSERIAL::__putc(int c, bool block) {
     return 0;
 }
 
-}; // namespace AjK ends
+} // namespace AjK ends

lib/MODSERIAL/RESIZE.cpp

@@ -120,4 +120,4 @@ MODSERIAL::upSizeBuffer(int size, IrqType type, bool memory_check)
     return Ok;
 }
 
-}; // namespace AjK ends
+} // namespace AjK ends

makefile

@@ -113,6 +113,15 @@ else
 	LINKER += -Llib/mbed/TARGET_LPC1768/TOOLCHAIN_GCC_ARM
 	LINKER += -lstdc++ -lsupc++ -lm -lc -lgcc -lnosys
 	LINKER += -lmbed
+	CPPSRC += lib/MODSERIAL/FLUSH.cpp
+	CPPSRC += lib/MODSERIAL/GETC.cpp
+	CPPSRC += lib/MODSERIAL/INIT.cpp
+	CPPSRC += lib/MODSERIAL/ISR_RX.cpp
+	CPPSRC += lib/MODSERIAL/ISR_TX.cpp
+	CPPSRC += lib/MODSERIAL/MODSERIAL_IRQ_INFO.cpp
+	CPPSRC += lib/MODSERIAL/MODSERIAL.cpp
+	CPPSRC += lib/MODSERIAL/PUTC.cpp
+	CPPSRC += lib/MODSERIAL/RESIZE.cpp
 	OBJ = lib/mbed/TARGET_LPC1768/TOOLCHAIN_GCC_ARM/board.o
 	OBJ += lib/mbed/TARGET_LPC1768/TOOLCHAIN_GCC_ARM/cmsis_nvic.o
 	OBJ += lib/mbed/TARGET_LPC1768/TOOLCHAIN_GCC_ARM/retarget.o