Happy little incident #4

[Mcu32]

by convention, printTo() returns the number of characters written.

not here:

cores/teensy4/IPAddress.cpp at master · PaulStoffregen/cores

Teensy Core Libraries for Arduino. Contribute to PaulStoffregen/cores development by creating an account on GitHub.

github.com

(may cause string corruption or wrong length later)

Solution:

size_t IPAddress::printTo(Print& p) const
{
    size_t n = 0;
    int i=0;
    while (1) {
        n += p.print(_address.bytes[i], DEC);
        if (++i >= 4) return n;
        n += p.write('.');
    }
}

use return n;

 

[Mcu32]

An other little issue (maybe):
IPAddress::fromString() accepts ("192.168.7.") or even ("...") - I'm not sure if this is wanted(?).