DHB improvements, possible bug #472
No reviewers
Labels
No Label
API Change
Breaking API Change
bug
build
cosmetics
Documentation
duplicate
feature
help wanted
hotfix
invalid
question
Refactor
Tests
wontfix
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: fsfw/fsfw#472
Loading…
Reference in New Issue
No description provided.
Delete Branch "eive:mueller/dhb-bugfixes-improvements"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
I found it a bit confusing after using DHB again after a long time how external commanding and the generation of verification messages is handled. These changes clarify the behaviour of DHB for external commanding a bit.
Additional diagnostic output for invalid commands was added as well.
Also, Iam not sure whether this is a bug, but a FDIR event will be trigged if
getSendSuccess
returnsNO_REPLY_EXPECTED
DHB improvementsto DHB improvements, possible bugI think there is a misunderstanding here:
NO_REPLY_EXPECTED
is not returned bygetSendSuccess()
but byenableReplyInReplyMap()
. Thus, it will not trigger an FDIR event as it is returned in theif
branch, while the check is located in theelse
path.result == NO_REPLY_EXPECTED
is handled correctly inreplyToCommand()
.OKay, I'll check this
#474 fixes actual problem, remaining is the misunderstanding (#wontfix) and a possible bug in eive FSW which needs to be confirmed.
Pull request closed