HD Sprint3 Branch Review

Activity

HD-SPR3-1 71

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author 1h 26m 23 Agreed. See HD-SPR4-1 review for fix.
    Reviewer - Complete 2h 3m 3 RESOLVED in CODE WALKTHROUGH
    pmontazemi  (deleted user)
    Reviewer completed
    Reviewer - Complete 1h 1 This is CLASS C code that needs to be reviewed in a group...
    lbaloa  (deleted user)
    Reviewer completed
    Reviewer - Complete 1h 29m 44 RESOLVED IN CODE WALKTHROUGH
    Total   5h 59m 71  
    #permalink

    Objectives

    There are no specific objectives for this review.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    lbaloa  (deleted user)

    We might want to change Contollers directory to Controllers

    We might want to change Contollers directory to Controllers

      lbaloa  (deleted user) marked as Resolved 22 Oct 19

    lbaloa  (deleted user)

    RESOLVED IN CODE WALKTHROUGH

    RESOLVED IN CODE WALKTHROUGH

    /App/Contollers/AlarmLamp.c Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Contollers/Buttons.c Changed 8
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Contollers/Buttons.h Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Drivers/CPLD.c Changed 10
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Drivers/CPLD.h Changed 3
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Services/CommBuffers.c Added 24
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Services/CommBuffers.h Added
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Services/MsgQueues.c Added 6
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Services/MsgQueues.h Added
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Services/SystemComm.c Added 12
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Services/SystemComm.h Added
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Services/SystemCommMessages.c Added
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Services/SystemCommMessages.h Added
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Services/Timers.c Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Services/WatchdogMgmt.c Changed 3
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Tasks/TaskBG.c Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Tasks/TaskGeneral.c Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /App/Common.h Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /Debug/App/Contollers/subdir_rules.mk Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /Debug/App/Drivers/subdir_rules.mk Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /Debug/App/Modes/subdir_rules.mk Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /Debug/App/Tasks/subdir_rules.mk Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /Debug/source/subdir_rules.mk Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /Debug/ccsObjs.opt Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /Debug/makefile Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /include/can.h Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /include/sys_vim.h Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /source/can.c Changed 3
    Open in IDE #permalink
    Repository HDProject does not exist
    /source/notification.c Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /source/rti.c Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /source/sys_link.cmd Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /source/sys_main.c Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /source/sys_startup.c Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /source/sys_vim.c Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /.cproject Changed
    Open in IDE #permalink
    Repository HDProject does not exist
    /HD.dil Changed
    Open in IDE #permalink
    Repository HDProject does not exist

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time