Turn NULLs into nullptr

Avoids overload problems with Callback(nullptr) versus Callback(fnptr).

Author: kjbracey

components/802.15.4_RF/atmel-rf-driver/source/NanostackRfPhyAtmel.cpp

@@ -524,7 +524,7 @@ static void rf_if_reset_radio(void)
 #else
     rf->spi.frequency(MBED_CONF_ATMEL_RF_LOW_SPI_SPEED);
 #endif
-    rf->IRQ.rise(0);
+    rf->IRQ.rise(nullptr);
     rf->RST = 1;
     ThisThread::sleep_for(2);
     rf->RST = 0;

components/wifi/esp8266-driver/ESP8266/ESP8266.cpp

@@ -41,7 +41,7 @@ ESP8266::ESP8266(PinName tx, PinName rx, bool debug, PinName rts, PinName cts)
     : _sdk_v(-1, -1, -1),
       _at_v(-1, -1, -1),
       _tcp_passive(false),
-      _callback(0),
+      _callback(),
       _serial(tx, rx, MBED_CONF_ESP8266_SERIAL_BAUDRATE),
       _serial_rts(rts),
       _serial_cts(cts),

components/wifi/esp8266-driver/ESP8266Interface.cpp

@@ -71,7 +71,7 @@ ESP8266Interface::ESP8266Interface()
       _connect_retval(NSAPI_ERROR_OK),
       _disconnect_retval(NSAPI_ERROR_OK),
       _conn_stat(NSAPI_STATUS_DISCONNECTED),
-      _conn_stat_cb(NULL),
+      _conn_stat_cb(),
       _global_event_queue(mbed_event_queue()), // Needs to be set before attaching event() to SIGIO
       _oob_event_id(0),
       _connect_event_id(0),
@@ -113,7 +113,7 @@ ESP8266Interface::ESP8266Interface(PinName tx, PinName rx, bool debug, PinName r
       _connect_retval(NSAPI_ERROR_OK),
       _disconnect_retval(NSAPI_ERROR_OK),
       _conn_stat(NSAPI_STATUS_DISCONNECTED),
-      _conn_stat_cb(NULL),
+      _conn_stat_cb(),
       _global_event_queue(mbed_event_queue()), // Needs to be set before attaching event() to SIGIO
       _oob_event_id(0),
       _connect_event_id(0),

drivers/SerialBase.h

@@ -216,7 +216,7 @@ class SerialBase : private NonCopyable<SerialBase> {
     /** Begin asynchronous write using 8bit buffer.
      *
      *  The write operation ends with any of the enabled events and invokes
-     *  registered callback function (which can be NULL to not receive callback at all).
+     *  registered callback function (which can be empty to not receive callback at all).
      *  Events that are not enabled by event argument are simply ignored.
      *  Operation has to be ended explicitly by calling abort_write() when
      *  no events are enabled.
@@ -233,7 +233,7 @@ class SerialBase : private NonCopyable<SerialBase> {
     /** Begin asynchronous write using 16bit buffer.
      *
      *  The write operation ends with any of the enabled events and invokes
-     *  registered callback function (which can be NULL to not receive callback at all).
+     *  registered callback function (which can be empty to not receive callback at all).
      *  Events that are not enabled by event argument are simply ignored.
      *  Operation has to be ended explicitly by calling abort_write() when
      *  no events are enabled.
@@ -256,7 +256,7 @@ class SerialBase : private NonCopyable<SerialBase> {
     /** Begin asynchronous reading using 8bit buffer.
      *
      *  The read operation ends with any of the enabled events and invokes registered
-     *  callback function (which can be NULL to not receive callback at all).
+     *  callback function (which can be empty to not receive callback at all).
      *  Events that are not enabled by event argument are simply ignored.
      *  Operation has to be ended explicitly by calling abort_read() when
      *  no events are enabled.
@@ -274,7 +274,7 @@ class SerialBase : private NonCopyable<SerialBase> {
     /** Begin asynchronous reading using 16bit buffer.
      *
      *  The read operation ends with any of the enabled events and invokes registered
-     *  callback function (which can be NULL to not receive callback at all).
+     *  callback function (which can be empty to not receive callback at all).
      *  Events that are not enabled by event argument are simply ignored.
      *  Operation has to be ended explicitly by calling abort_read() when
      *  no events are enabled.

drivers/internal/PolledQueue.h

@@ -42,7 +42,7 @@ class PolledQueue: public TaskQueue {
      *
      *  @param cb Callback called when dispatch needs to be called
      */
-    PolledQueue(mbed::Callback<void()> cb = NULL);
+    PolledQueue(mbed::Callback<void()> cb = nullptr);
 
     virtual ~PolledQueue();
 

drivers/internal/USBDevice.h

@@ -147,7 +147,7 @@ class USBDevice: public  USBPhyEvents {
     * @param callback Method pointer to be called when a packet is transferred
     * @returns true if successful, false otherwise
     */
-    bool endpoint_add(usb_ep_t endpoint, uint32_t max_packet, usb_ep_type_t type, mbed::Callback<void()> callback = NULL);
+    bool endpoint_add(usb_ep_t endpoint, uint32_t max_packet, usb_ep_type_t type, mbed::Callback<void()> callback = nullptr);
 
     /**
     * Add an endpoint

drivers/source/CAN.cpp

@@ -56,7 +56,7 @@ CAN::~CAN()
 
     // Detaching interrupts releases the sleep lock if it was locked
     for (int irq = 0; irq < IrqCnt; irq++) {
-        attach(NULL, (IrqType)irq);
+        attach(nullptr, (IrqType)irq);
     }
     can_irq_free(&_can);
     can_free(&_can);
@@ -147,7 +147,7 @@ void CAN::attach(Callback<void()> func, IrqType type)
         if (_irq[(CanIrqType)type]) {
             sleep_manager_unlock_deep_sleep();
         }
-        _irq[(CanIrqType)type] = NULL;
+        _irq[(CanIrqType)type] = nullptr;
         can_irq_set(&_can, (CanIrqType)type, 0);
     }
     unlock();

drivers/source/InterruptIn.cpp

@@ -26,8 +26,8 @@ namespace mbed {
 //       constructor, with a default value for the PinMode.
 InterruptIn::InterruptIn(PinName pin) : gpio(),
     gpio_irq(),
-    _rise(NULL),
-    _fall(NULL)
+    _rise(),
+    _fall()
 {
     // No lock needed in the constructor
     irq_init(pin);
@@ -37,8 +37,8 @@ InterruptIn::InterruptIn(PinName pin) : gpio(),
 InterruptIn::InterruptIn(PinName pin, PinMode mode) :
     gpio(),
     gpio_irq(),
-    _rise(NULL),
-    _fall(NULL)
+    _rise(),
+    _fall()
 {
     // No lock needed in the constructor
     irq_init(pin);
@@ -76,7 +76,7 @@ void InterruptIn::rise(Callback<void()> func)
         _rise = func;
         gpio_irq_set(&gpio_irq, IRQ_RISE, 1);
     } else {
-        _rise = NULL;
+        _rise = nullptr;
         gpio_irq_set(&gpio_irq, IRQ_RISE, 0);
     }
     core_util_critical_section_exit();
@@ -89,7 +89,7 @@ void InterruptIn::fall(Callback<void()> func)
         _fall = func;
         gpio_irq_set(&gpio_irq, IRQ_FALL, 1);
     } else {
-        _fall = NULL;
+        _fall = nullptr;
         gpio_irq_set(&gpio_irq, IRQ_FALL, 0);
     }
     core_util_critical_section_exit();

drivers/source/SPI.cpp

@@ -151,14 +151,14 @@ SPI::~SPI()
     lock();
     /* Make sure a stale pointer isn't left in peripheral's owner field */
     if (_peripheral->owner == this) {
-        _peripheral->owner = NULL;
+        _peripheral->owner = nullptr;
     }
     unlock();
 }
 
 SPI::spi_peripheral_s *SPI::_lookup(SPI::SPIName name)
 {
-    SPI::spi_peripheral_s *result = NULL;
+    SPI::spi_peripheral_s *result = nullptr;
     core_util_critical_section_enter();
     for (int idx = 0; idx < _peripherals_used; idx++) {
         if (_peripherals[idx].name == name) {

drivers/source/Serial.cpp

@@ -28,11 +28,11 @@ Serial::Serial(const serial_pinmap_t &static_pinmap, const char *name, int baud)
 {
 }
 
-Serial::Serial(PinName tx, PinName rx, int baud): SerialBase(tx, rx, baud), Stream(NULL)
+Serial::Serial(PinName tx, PinName rx, int baud): SerialBase(tx, rx, baud), Stream()
 {
 }
 
-Serial::Serial(const serial_pinmap_t &static_pinmap, int baud): SerialBase(static_pinmap, baud), Stream(NULL)
+Serial::Serial(const serial_pinmap_t &static_pinmap, int baud): SerialBase(static_pinmap, baud), Stream()
 {
 }