diff options
author | Kevin O'Connor <kevin@koconnor.net> | 2017-06-20 12:09:55 -0400 |
---|---|---|
committer | Kevin O'Connor <kevin@koconnor.net> | 2017-06-29 13:33:59 -0400 |
commit | c1bd628ce524f52851cf2bbcb4bc8bbc373dcd6b (patch) | |
tree | 4b3705dbf34c7f8ec0f52a7fadae17f43031d31b /src | |
parent | 849096d5f35185b082571665ca590fed9c35a073 (diff) | |
download | kutter-c1bd628ce524f52851cf2bbcb4bc8bbc373dcd6b.tar.gz kutter-c1bd628ce524f52851cf2bbcb4bc8bbc373dcd6b.tar.xz kutter-c1bd628ce524f52851cf2bbcb4bc8bbc373dcd6b.zip |
command: Directly call command_sendf() for ack/nak messages
Don't use the sendf() macro for ack and nak messages - directly call
the command_sendf() code instead.
Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
Diffstat (limited to 'src')
-rw-r--r-- | src/command.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/command.c b/src/command.c index 62db0bfd..af84539f 100644 --- a/src/command.c +++ b/src/command.c @@ -222,6 +222,11 @@ command_lookup_parser(uint8_t cmdid) return &command_index[cmdid]; } +// Empty message (for ack/nak transmission) +const struct command_encoder encode_acknak PROGMEM = { + .max_size = MESSAGE_MIN, +}; + enum { CF_NEED_SYNC=1<<0, CF_NEED_VALID=1<<1 }; // Find the next complete message. @@ -256,7 +261,7 @@ command_find_block(char *buf, uint8_t buf_len, uint8_t *pop_count) goto nak; } next_sequence = ((msgseq + 1) & MESSAGE_SEQ_MASK) | MESSAGE_DEST; - sendf(""); // An empty message with a new sequence number is an ack + command_sendf(&encode_acknak); return 1; need_more_data: @@ -282,7 +287,7 @@ need_sync: ; return -1; sync_state |= CF_NEED_VALID; nak: - sendf(""); // An empty message with a duplicate sequence number is a nak + command_sendf(&encode_acknak); return -1; } |