Two protected areas in Stream.h

[SummerSun]

Is it a typo? The first one should be private according to the comments.

class Stream : public Print
{

protected:

    unsigned long _timeout;      // number of milliseconds to wait......
    unsigned long _startMillis;  // used for timeout measurement
    int timedRead();    // private method to read stream with timeout
    int timedPeek();    // private method to peek stream with timeout
public:
....
...

protected:

        long parseInt(char skipChar); // as above but the given skipChar is ignored
        // as above but the given skipChar is ignored
        // this allows format characters (typically commas) in values to be ignored

        float parseFloat(char skipChar);  // as above but the given skipChar is ignored

[cousteaulecommandant]

This is intentional. Apparently it used to be private, but it was changed to protected for optimization reasons (see 99f2a27).
Maybe the comments should have been changed accordingly though.

Tip: use the Blame button when browsing a file on Github (https://github.com/arduino/Arduino/blob/master/hardware/arduino/avr/cores/arduino/Stream.h) to see which was the last commit to affect each line of the file.

[matthijskooijman]

So the only change needed is to fix the comments. @cmaglie, perhaps you could fix this? Or make me collaborator on the sam/samd repositories too so I can push a simple fix?

[per1234]

Closed by:
#6464
arduino/ArduinoCore-samd#244
arduino/ArduinoCore-sam#37