-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Work/dp catalog #2713
base: devel
Are you sure you want to change the base?
Work/dp catalog #2713
Conversation
this->m_xmitCmdWait = false; | ||
this->cmdResponse_out(this->m_xmitOpCode, this->m_xmitCmdSeq, Fw::CmdResponse::OK); | ||
} | ||
DpCatalog::DpBtreeNode* DpCatalog::findNextTreeNode() { |
Check notice
Code scanning / CodeQL
Long function without assertion
|
||
// traverse the tree, finding nodes in order. Max iteration of the loop | ||
// would be the number of records in the tree | ||
for (FwSizeType record = 0; record < this->m_numDpRecords; record++) { |
Check notice
Code scanning / CodeQL
Use of basic integral type
|
Change Description
A description of the changes contained in the PR.
Rationale
A rationale for this change. e.g. fixes bug, or most projects need XYZ feature.
Testing/Review Recommendations
Fill in testing procedures, specific items to focus on for review, or other info to help the team verify these changes are flight-quality.
Future Work
Note any additional work that will be done relating to this issue.