Refactor to use getHDFAttributes function #6
No reviewers
Labels
No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: nqrduck/limedriver#6
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "hdfattr_function_fix"
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?
Moved the constuction of the HDF attribute array to a dedicated function so we can use it from other places than main() in the future.
Fixed merge issues of #4
With the current implementation the RX data stream is empty.
Let's dive into this. This is the content of the array with the code from this branch:
And this is from main:
Do your elven eyes spy a difference?
Indeed.
This pulls request shall not passs in it's current form!
With the possible exception of the below, it does seem to be an initialization issue, though. It reads values where it should be reading 0.
This is still weird:
Just checked, when I initialize the struct as
LimeConfig_t LimeCfg{};
(rather thanLimeConfig_t LimeCfg;
) ininitializeLimeConfig()
, that gets rid of that part of the problem. Now I'm trying to figure out the other.The other issue is already present here, not only in getHDFAttributes.
7414d1490c/src/limedriver.cpp (L427-L428)