Closed lunix80 closed 7 years ago
Looks good.
I don't think you need those parentheses in your regex as that would just create a grouping and you never do a back reference to look at the grouping. Doesn't really change the behavior though, just one extra memory allocation.
Pull request merged and 1.94 released. https://github.com/chilipeppr/serial-port-json-server/releases/tag/v1.94
Good work on changes.
On Mon, Jan 16, 2017 at 3:29 PM, Luca notifications@github.com wrote:
@lunix80 commented on this pull request.
Buffer decreased from 127 to 125 bytes to be safe
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/chilipeppr/serial-port-json-server/pull/23#pullrequestreview-16898963, or mute the thread https://github.com/notifications/unsubscribe-auth/AHidbYMQ2M8AJ0Rb-D0KWPuF_6i9KHtUks5rS_0_gaJpZM4Lk-ei .
Adding extra regex match to send directly to the serial port new real-time commands of grbl v1.1