big but minor changes: remove lots of duplicate code, move trivial calls to .h to give gcc a chance to optimize them

Author: d-a-v

cores/esp8266/HardwareSerial.cpp

@@ -55,10 +55,8 @@ void HardwareSerial::end()
         uart_set_debug(UART_NO);
     }
 
-    if (_uart) {
-        uart_uninit(_uart);
-        _uart = NULL;
-    }
+    uart_uninit(_uart);
+    _uart = NULL;
 }
 
 size_t HardwareSerial::setRxBufferSize(size_t size){
@@ -70,30 +68,6 @@ size_t HardwareSerial::setRxBufferSize(size_t size){
     return _rx_size;
 }
 
-void HardwareSerial::swap(uint8_t tx_pin)
-{
-    if(!_uart) {
-        return;
-    }
-    uart_swap(_uart, tx_pin);
-}
-
-void HardwareSerial::set_tx(uint8_t tx_pin)
-{
-    if(!_uart) {
-        return;
-    }
-    uart_set_tx(_uart, tx_pin);
-}
-
-void HardwareSerial::pins(uint8_t tx, uint8_t rx)
-{
-    if(!_uart) {
-        return;
-    }
-    uart_set_pins(_uart, tx, rx);
-}
-
 void HardwareSerial::setDebugOutput(bool en)
 {
     if(!_uart) {
@@ -113,16 +87,6 @@ void HardwareSerial::setDebugOutput(bool en)
     }
 }
 
-bool HardwareSerial::isTxEnabled(void)
-{
-    return _uart && uart_tx_enabled(_uart);
-}
-
-bool HardwareSerial::isRxEnabled(void)
-{
-    return _uart && uart_rx_enabled(_uart);
-}
-
 int HardwareSerial::available(void)
 {
     int result = static_cast<int>(uart_rx_available(_uart));
@@ -132,27 +96,6 @@ int HardwareSerial::available(void)
     return result;
 }
 
-int HardwareSerial::peek(void)
-{
-    // this may return -1, but that's okay
-    return uart_peek_char(_uart);
-}
-
-int HardwareSerial::read(void)
-{
-    // this may return -1, but that's okay
-    return uart_read_char(_uart);
-}
-
-int HardwareSerial::availableForWrite(void)
-{
-    if(!_uart || !uart_tx_enabled(_uart)) {
-        return 0;
-    }
-
-    return static_cast<int>(uart_tx_free(_uart));
-}
-
 void HardwareSerial::flush()
 {
     if(!_uart || !uart_tx_enabled(_uart)) {
@@ -165,33 +108,9 @@ void HardwareSerial::flush()
     delayMicroseconds(11000000 / uart_get_baudrate(_uart) + 1);
 }
 
-size_t HardwareSerial::write(uint8_t c)
-{
-    return uart_write_char(_uart, c);
-}
-
-size_t HardwareSerial::write(const uint8_t *buffer, size_t size)
-{
-    return uart_write(_uart, (const char*)buffer, size);
-}
-
-int HardwareSerial::baudRate(void)
-{
-    // Null pointer on _uart is checked by SDK
-    return uart_get_baudrate(_uart);
-}
-
-
-HardwareSerial::operator bool() const
-{
-    return _uart != 0;
-}
-
-
 #if !defined(NO_GLOBAL_INSTANCES) && !defined(NO_GLOBAL_SERIAL)
 HardwareSerial Serial(UART0);
 #endif
 #if !defined(NO_GLOBAL_INSTANCES) && !defined(NO_GLOBAL_SERIAL1)
 HardwareSerial Serial1(UART1);
 #endif
-

cores/esp8266/HardwareSerial.h

@@ -93,26 +93,50 @@ class HardwareSerial: public Stream
     {
         swap(1);
     }
-    void swap(uint8_t tx_pin);    //toggle between use of GPIO13/GPIO15 or GPIO3/GPIO(1/2) as RX and TX
+    void swap(uint8_t tx_pin)    //toggle between use of GPIO13/GPIO15 or GPIO3/GPIO(1/2) as RX and TX
+    {
+        uart_swap(_uart, tx_pin);
+    }
 
     /*
      * Toggle between use of GPIO1 and GPIO2 as TX on UART 0.
      * Note: UART 1 can't be used if GPIO2 is used with UART 0!
      */
-    void set_tx(uint8_t tx_pin);
+    void set_tx(uint8_t tx_pin)
+    {
+        uart_set_tx(_uart, tx_pin);
+    }
 
     /*
      * UART 0 possible options are (1, 3), (2, 3) or (15, 13)
      * UART 1 allows only TX on 2 if UART 0 is not (2, 3)
      */
-    void pins(uint8_t tx, uint8_t rx);
+    void pins(uint8_t tx, uint8_t rx)
+    {
+        uart_set_pins(_uart, tx, rx);
+    }
 
     int available(void) override;
-    int peek(void) override;
-    int read(void) override;
-    int availableForWrite(void);
+
+    int peek(void) override
+    {
+        // this may return -1, but that's okay
+        return uart_peek_char(_uart);
+    }
+    int read(void) override
+    {
+        // this may return -1, but that's okay
+        return uart_read_char(_uart);
+    }
+    int availableForWrite(void)
+    {
+        return static_cast<int>(uart_tx_free(_uart));
+    }
     void flush(void) override;
-    size_t write(uint8_t) override;
+    size_t write(uint8_t c) override
+    {
+        return uart_write_char(_uart, c);
+    }
     inline size_t write(unsigned long n)
     {
         return write((uint8_t) n);
@@ -129,13 +153,27 @@ class HardwareSerial: public Stream
     {
         return write((uint8_t) n);
     }
-    size_t write(const uint8_t *buffer, size_t size);
-    operator bool() const;
-
+    size_t write(const uint8_t *buffer, size_t size)
+    {
+        return uart_write(_uart, (const char*)buffer, size);
+    }
+    operator bool() const
+    {
+        return _uart != 0;
+    }
     void setDebugOutput(bool);
-    bool isTxEnabled(void);
-    bool isRxEnabled(void);
-    int  baudRate(void);
+    bool isTxEnabled(void)
+    {
+        return uart_tx_enabled(_uart);
+    }
+    bool isRxEnabled(void)
+    {
+        return _uart && uart_rx_enabled(_uart);
+    }
+    int baudRate(void)
+    {
+        return uart_get_baudrate(_uart);
+    }
 
     bool hasOverrun(void)
     {