search

thijse / Arduino-CmdMessenger

CmdMessenger Communication library for Arduino & .NET
Other
208 stars 87 forks source link

Remove deps #21

Closed Palatis closed 6 years ago

Palatis commented 7 years ago

remove unnecessary dependencies:

and remove System.Windows.Forms from CommandMessenger, keep it a pure library project with minimum dependency.

Also, because we removed System.Windows.Forms from CommandMessenger, create a CommandMessenger.Utils project with a helper class to check for InvokeRequired, and Invoke() the work on proper thread.

Palatis commented 7 years ago

pretty much what was discussed in https://github.com/thijse/Arduino-CmdMessenger/pull/18, without the .Net 3.5 backport.

Palatis commented 7 years ago

okay, unity fails to read the first character of each transmission, I had to workaround the problem by sending an extra newline before every command.

that's a dirty workaround, i think i'll keep it in my own repo.