action helper nullptr check #454
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#454
Loading…
Reference in New Issue
No description provided.
Delete Branch "mueller/action-printout"
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?
Target is #445 for easier comparison now, might be retargeted.
Initialize has checked that the pointer is valid before (as is the main purpose of
initialize()
, checking the settings).While I agree that forgetting to call
initialize()
is an annoying and hard to find bug, I am not sure we should enforce it by doing redundant checks throughout the code.I think this was forgotten because it was overriden in the
ExtendedControllerBase
..nullptr
checks are cheap, so I introduced this because the software crashed without any warning.. but I'm not sure whether it is worth it.. It just improved the capability of the FSFW to catch configuration errorsPull request closed