Why read byte by byte when it is possible to read all the bytes in can\interfaces\slcan.py?

[silverv]

@felixdivo @mikisama

python-can/can/interfaces/slcan.py

Line 164 in af52622

new_byte = self.serialPortOrig.read(size=1)

Is there a specific reason for size=1? Before size=1, in git blame, it was not set and the default was still 1.
As an alternative, that also works, it is possible to read the bytes that are waiting to be read in one go, and hopefully delay less:
new_bytes = self.serialPortOrig.read(size=self.serialPortOrig.in_waiting)
I am in the process of measuring the changed delays.

[felixdivo]

There was a long discussion in #1490.

🤷 I'm not into that, so I'll retreat here.

[silverv]

In continuation to #1490 I created #1572