RESOLVED IN CODE WALKTHROUGH
Use BOOL instead of U32 for sizeof checks since it is a BOOL that you are copying into. Same size, but want to be consistent.
RESOLVED in CODE WALKTHROUGH.
Add TODO to comment.
Change was made to get around earlier issues. Need discussion on whether or not to change back.
Removed.
Done.
Use BOOL. And add a blank line after. And make following condition explicit (i.e. != TRUE).
Remove blank line.
Restore blank line between functions.
Remove commented out line of code.
Fixed. Thanks!
Add a space between "return" and "(".
Do we need counters since we're using persistent alarms?
Will address in DEN-13460.
Thanks, Fixed.
Removed blank lilne.
This code will be updated to be consistent with the getRsrvrFillStatus from FLUSH mode.
/*@)/ appears to be the last line in all files? Should it put back in. Must be a doxygen command.
Yes. And don't mark replies as requiring resolution.
Removed in ed6af22
Corrected.
Bill, I was just asking why there was a blank line added before final } of last function. The doxygen end group stuff should stay. I will generally add my comments directly below the issue, not above.
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.