fsfw/fsfw
fsfw/fsfw
10
3
Fork 6
Code Issues 31 Pull Requests 4 Releases 10 Wiki Activity

more printouts for rejected TC packets #505

Merged
mohr merged 5 commits from eive/fsfw:mueller/packet-check-printout into development 2021-10-18 14:44:40 +02:00
Conversation 0 Commits 5 Files Changed 1 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 113c992f99 - Show all commits

2
src/fsfw/tcdistribution/PUSDistributor.cpp
View File

@ -29,7 +29,7 @@ PUSDistributor::TcMqMapIter PUSDistributor::selectDestination() {
tcStatus = checker.checkPacket(currentPacket);
if(tcStatus != HasReturnvaluesIF::RETURN_OK) {
#if FSFW_VERBOSE_LEVEL >= 1
std::string keyword;
const char* keyword = nullptr;
mohr marked this conversation as resolved Outdated
mohr commented 2021-10-15 16:51:08 +02:00
Outdated
Review
Copy Link

As we try to not use allocating classes in the framework during runtime, I do not see a reason not to use a char Pointer here.

As we try to not use allocating classes in the framework during runtime, I do not see a reason not to use a `char` Pointer here.
muellerr commented 2021-10-15 18:35:14 +02:00
Outdated
Review
Copy Link

We should provide a custom allocator in the future so that users can use classes like std::vector and std::string without issues in the future. Another solution would be to use etl::vector and etl::string which do not allocate dynamically. cmake allows good dependency management for this. But this is a topic for another issue.

I'll change the code to use a preallocated array.

We should provide a custom allocator in the future so that users can use classes like `std::vector` and `std::string` without issues in the future. Another solution would be to use `etl::vector` and `etl::string` which do not allocate dynamically. cmake allows good dependency management for this. But this is a topic for another issue. I'll change the code to use a preallocated array.
mohr commented 2021-10-18 10:55:48 +02:00
Outdated
Review
Copy Link

If you change the init value to "unnamed error" from nullptr, I'll be really happy :)

Fully aggree re allocators and etl, I tend towards etl, as we already started implementing something like it in /containers. But you are right, this is a topic for anopther issue.

If you change the init value to `"unnamed error"` from `nullptr`, I'll be really happy :) Fully aggree re allocators and etl, I tend towards etl, as we already started implementing something like it in `/containers`. But you are right, this is a topic for anopther issue.
muellerr commented 2021-10-18 13:58:43 +02:00
Outdated
Review
Copy Link

done

done
if(tcStatus == TcPacketCheck::INCORRECT_CHECKSUM) {
keyword = "checksum";
}