don't generate acks for locally sourced msgs

This commit is contained in:
Kevin Hester 2021-03-05 12:37:26 +08:00
parent 8739469db3
commit c9c44a934d
2 changed files with 7 additions and 1 deletions

View File

@ -34,6 +34,8 @@ You probably don't care about this section - skip to the next one.
* DONE combine acks and responses in a single message if possible (do routing plugin LAST and drop ACK if someone else has already replied)
* DONE don't send packets we received from the phone BACK TOWARDS THE PHONE (possibly use fromnode 0 for packets the phone sends?)
* fix 1.1.50 android debug panel display
* test android channel setting
* release to users
* DONE warn in android app about unset regions
* DONE use set-channel from android
* DONE add gui in android app for setting region

View File

@ -109,7 +109,11 @@ void setReplyTo(MeshPacket *p, const MeshPacket &to)
{
assert(p->which_payloadVariant == MeshPacket_decoded_tag); // Should already be set by now
p->to = getFrom(&to);
p->want_ack = to.want_ack;
// No need for an ack if we are just delivering locally (it just generates an ignored ack)
p->want_ack = (to.from != 0) ? to.want_ack : false;
if(p->priority == MeshPacket_Priority_UNSET)
p->priority = MeshPacket_Priority_RELIABLE;
p->decoded.request_id = to.id;
}