Skip to content

Conversation

pull[bot]
Copy link

@pull pull bot commented Jan 20, 2020

See Commits and Changes for more details.


Created by pull[bot]. Want to support this open source service? Please star it : )

me-no-dev and others added 9 commits January 20, 2020 15:18
* Add Uri with support for staticUri, regexUri and globUri * Add newline to end of files * Update example * Suppress gcc warnings (unused params)
Hi, I have added a variant for the LilyGo TTGO LoRa32-OLED V2, which is similar to the LilyGo TTGO LoRa32-OLED V1 but with a different pinout. This will be useful for PlatformIO users, specially because I am also adding the board there as well. Cheers.
…3487) If two headers with overlapping names are added while replace == true, like in: ```cpp http.addHeader("api_token", "pMXFOLpinQqajaRQJYMeWObg2XYmcX1"); http.addHeader("token", "1234"); ``` then replacing went wrong. This is fixed with this PR.
* WString explicit converters to reduce Flash size This is a port from the same patch for ESP8266: https://github.com/esp8266/Arduino/pull/6759/files
* BLE: reception support multiple service data * fix prev commit, i not inicialized
With the >= used let's say you have four RMT inputs, each using 2 channels wide for their receive buffer. This is 4*2 = 8 buffers which is the number of hardware buffers (MAX_CHANNELS). But, for the fourth input the starting buffer will be 6 (this is correct, the buffers used for each input are 0-1, 2-3, 4-5, 6-7). But, 6+2 = 8 which is MAX_CHANNELS. This is valid but the >= would match against it and abort. It is correct to only abort if the value i+j is only greater than MAX_CHANNELS. Thus, a simple one character fix. Delete the equals sign.
Added pinout for TTGO T Journal device and improved the intent of comments.
@pull pull bot added the ⤵️ pull label Jan 20, 2020
@pull pull bot merged commit 5443d7c into nazt:master Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
8 participants