[riot-notifications] [RIOT-OS/RIOT] cpu/esp_common: Add missing includes and ISO-C++ compatibility (#15735)

benpicco notifications at github.com
Sun Jan 10 17:45:54 CET 2021


@benpicco commented on this pull request.



> @@ -103,10 +103,10 @@
  */
 typedef struct
 {
-    uint8_t* key;         /**< key of type uint8_t [16] or NULL (no encryption) */
-    uint32_t scan_period; /**< Period at which the node scans for other nodes */
-    char*    softap_pass; /**< Passphrase used for the SoftAP interface */
-    uint8_t  channel;     /**< Channel used for ESP-NOW nodes */
+    const uint8_t* key;      /**< key of type uint8_t [16] or NULL (no encryption) */
+    uint32_t scan_period;    /**< Period at which the node scans for other nodes */
+    const char* softap_pass; /**< Passphrase used for the SoftAP interface */
+    uint8_t  channel;        /**< Channel used for ESP-NOW nodes */

might as well remove the double white space - or align the names again :wink: 

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15735#pullrequestreview-564880723
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210110/495db2f6/attachment-0001.htm>


More information about the notifications mailing list