Remove carriage returns when parsing received data#200
Conversation
|
I think modifying this PR to use an Also a boolean value to ignore CR would preserve the use case here when set to false. |
|
Okay, this "works on my machine" 😉 It definitely warrants a review by more familiar eyes in case I missed something important when setting up a persistant variable. Documentation of a reproducible example to follow in #199 (might be a few days if it's left up to me). Haven't yet added the |
|
Hi, looks good to me! I think that this is the most elegant solution for this issue without breaking support for |
Fixes #199
Don't merge this yet, might try a sneaky switch case first