Arduino networking code
Merge request reports
Activity
Fix: If command overruns buffer, there can never be a \n so that buffer stays forever filled (it is cleared on reconnect; this is not an option for serial. Also, out-of-bounds write occurs as far as I can tell, so it's safe, just annoying.).
EDIT: finished
Edited by Kyrre Ness Sjobaekadded 1 commit
- d2efe175 - Fix inconsistencies in newline handling in parser.
added 2 commits
added 1 commit
- 93ca3ee9 - Grabber support + overload bufferWrite + flash-strings to bufferWrite
Temperature and grabber should in principle work.
TODO in system:
Avoid race conditions:
- If executing a "racy" command (i.e. moving an actuator, not just displaying data from RAM), halt parsing after parsing and queuing that command (with clean buffers etc.), run the commands, then continue parsing. finished
- Only allow one single client to take control at a time? Special command to "take control" after connecting, (defaulting to serial?). Message to tell who took control, and message to new clients at beginning of connection to allow self-identification.
Configuration:
- Move configs to header file, i.e. not in the main source file. finished
- Unneccessary "in use" flag for network connection slots? finished
Edited by Kyrre Ness Sjobaekadded 1 commit
- a35a4c54 - Move deployer's config into separate .h file
added 1 commit
- 91ed13bc - Flashify some strings; every byte counts when all we have is 8k...
added 1 commit
- 0039bb16 - Output line ending consistency (\n everywhere, no more \r\n which is what println() makes)
Please register or sign in to reply