Member attachInterrupt (uint8_t pin, InterruptCallback callback, GPIO_INT_TYPE mode): Add GPIO_INT_TYPE documentation - is this in SDK?
Member CircularBuffer::getStreamType () const override: Return value of IDataSourceStream:getStreamType base class function should be of type StreamType, e.g. eSST_User
Member CircularBuffer::readMemoryBlock (char *data, int bufSize) override: Should IDataSourceStream::readMemoryBlock return same data type as its bufSize param?
Member DS18S20::IsValidTemperature (uint8_t): Initial code review suggests this is set once but never rechecked so may not be accurate
Member fileClearLastError (file_t fd): Why does fileClearLastError have unused fd parameter?
Member fileLastError (file_t fd): Why does fileLastError have unused fd parameter?
Member fileStats (const String &name, spiffs_stat *stat): Document the return value of fileStats
Member fileStats (file_t file, spiffs_stat *stat): Document the return value of fileStats
Class HttpHeaders: add name and/or value escaping
Class HttpParams: values stored in escaped form, unescape return value and escape provided values. Revise HttpBodyParser.cpp as it will no longer do this job.
Member IDataSourceStream::readMemoryBlock (char *data, int bufSize)=0: Should IDataSourceStream::readMemoryBlock return same data type as its bufSize param?
Class IDelegateCaller< ReturnType, ParamsList >: Provide more informative brief description of IDelegateCaller
Member MIME_TYPE_MAP (XX): Consider using sz-strings for file extension to enable matching to alternative file extensions
Member MqttClient::publishWithQoS (const String &topic, const String &message, int QoS, bool retained=false, MqttMessageDeliveredCallback onDelivery=nullptr): deprecate: Use publish(const String& topic, const String& message, uint8_t flags = 0) instead. If you want to have a callback that should be triggered on successful delivery of messages then use setEventHandler(MQTT_TYPE_PUBACK, youCallback) instead.
Member MqttClient::setCallback (MqttStringSubscriptionCallback subscriptionCallback=nullptr): deprecate: Use setEventHandler(MQTT_TYPE_PUBLISH, MqttDelegate handler) instead.
Member MqttClient::setWill (const String &topic, const String &message, int QoS, bool retained=false): deprecate: Use setWill(const String& topic, const String& message,uint8_t flags) instead
Member pinMode (uint16_t pin, uint8_t mode): ESP8266 supports pull-down on GPIO 0-15
Member RF24::openReadingPipe (uint8_t number, uint64_t address): Enforce the restriction that pipes 1-5 must share the top 32 bits
Member RF24::setPayloadSize (uint8_t size): Implement variable-sized payloads feature
Member SystemClockClass::setTimeZoneOffset (int seconds): Why does this need to be set to 2 for UK during winter?
Member TaskCallback )(uint32_t param): Integrate delegation into callbacks
Member WDTClass::alive (): Define the WDT period (how long before it triggers)
Module wifi_ap: How is wifi access point dhcp controlled?