This is a list of all comments for HD-DEN-13141-6.

Review Summary: No summary


----------------------------------------
File: firmware/App/Modes/ModeInitPOST.c
Revision Comment by Sean Nash on 21 July 2022, 17:08 
https://devapps.diality.us/cru/HD-DEN-13141-6#c13305

  Why?

  Reply by wbracken on 15 August 2022, 17:44
  > You mean the blank line?

    Reply by Sean Nash on 16 August 2022, 09:38
    > Yes.  And don't mark replies as requiring resolution.

    Reply by wbracken on 16 August 2022, 10:06
    > Removed blank lilne.

      Reply by Sean Nash on 17 August 2022, 16:04
      > RESOLVED in CODE WALKTHROUGH.

Revision Comment by Sean Nash on 21 July 2022, 17:08 
https://devapps.diality.us/cru/HD-DEN-13141-6#c13306

  Why?

  Reply by wbracken on 15 August 2022, 17:50
  > Not sure why that's there.  Will remove it

  Reply by wbracken on 16 August 2022, 14:44
  > /**@)*/ appears to be the last line in all files?  Should it
  > put back in.  Must be a doxygen command.

    Reply by Sean Nash on 17 August 2022, 14:40
    > 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.

      Reply by Sean Nash on 19 August 2022, 15:30
      > RESOLVED in CODE WALKTHROUGH.


---

ID: HD-DEN-13141-6 https://devapps.diality.us/cru/HD-DEN-13141-6

Title: HD-DEN-13141_Miscellaneous HD Issues BB
Statement of Objectives:


State: Closed

Summary: 

Author: wbracken
Moderator: wbracken
Reviewers: (2 active, 2 completed*)
     Sean Nash   (*)
     Dara Navaei   (*)
     Michael Garthwaite
     Dong Nguyen