Hi Daniel, I actually caught that already: the diff I included before was stale (from a scroll-up in the terminal!).
Thanks! Brian > On Apr 26, 2019, at 11:45 AM, Daniel Fuchs <[email protected]> wrote: > > Looks good Brian! > > nit: > > > + reader.reset(); // reset to position after '\n' > > actually it's 'after '\r'' now. > > No need for new review!
