{"text_format":"3","workbook":"xlsx","n_sheets":3,"sheets":["ReadMe","CheckList","Criteria"],"not_rendered":{"defined_names":6,"comments":1},"read":"whole","date_system":"1900"}

{"sheet":"ReadMe","index":0,"n_rows":13,"n_cols":1,"rows_kept":13}
["This code review checklist is designed to be used either as the checklist of a guidebook for the checklist."]
[]
["Each question has assoicated help information embedded in a comment associated with that question. When printing you can either include a print of those comments or print without those comments. The template includes printing of those comments."]
[]
["How to remove comment printing - to print just the checklist"]
["File -> Page Setup"]
["Sheet tab"]
["Print options (middle of window) Comment pulldown select \"(None)\""]
[]
["How to add comment printing - to print as a guidebook"]
["File -> Page Setup"]
["Sheet tab"]
["Print options (middle of window) Comment pulldown select \"At end of sheet\""]

{"sheet":"CheckList","index":1,"n_rows":69,"n_cols":5,"rows_kept":69,"merged":[[0,0,0,4],[9,0,9,4],[13,0,13,4],[19,0,19,4],[22,0,22,4],[29,0,29,4],[41,0,41,4],[47,0,47,4],[52,0,52,4],[57,0,57,4],[59,0,59,4],[63,0,63,4],[66,0,66,4]]}
["Component Code Peer Review - Version 1.1"]
["Software Component Name","ActiveRateGroup"]
["Component Developer","T. Canham"]
["Component Peer Reviewers","Watney, Reder, Nowick, Wilf"]
["Software Type","Re-engineer (Functional)   - Code changes planned (<20%)"]
["Review Date","2015-08-18"]
[]
[]
["Question Number","Subject","Developer Assessment","Peer Assessment","Comments"]
[" Prior Phase Completion"]
[1,"Have all action items from the design review been closed?","Yes","Yes"]
[{"$f":"A11+1","v":2},"Do all non-closed actions items have NO code impact?","n/a","n/a"]
[{"$f":"A12+1","v":3},"Are all open SPRs for this module incorporated into the code?","n/a","n/a"]
[" Package Completeness"]
[{"$f":"A13+1","v":4},"Has the entire component been committed to the remote delivery branch of the Git repository.","Yes","Yes"]
[{"$f":"A15+1","v":5},"Does the reviewed software have a git hash code of the last commit?","Yes","Yes","Reviewing commit 3de1fff"]
[6,"Does the unit compile without warnings for all required targets?","Yes","Yes"]
[7,"Does the review package contain the latest SLOC count?","Yes","Yes",{"$f":"13+137+59+66+486+90","v":851}]
[8,"Have the requirements been identified that are satisfied by this component implementation?","Yes","Yes"]
["Coding Standards"]
[9,"Does the code adhere to the set of required coding standards?","No","No","Rule 17"]
[10,"Does the code pass all static analysis with no warnings?","No","No","Autocoder has one issue; ticket written"]
["Code Architecture"]
[11,"Does each function/method implement a single activity?","Yes","Yes"]
[12,"Has a consistent and standard function banner been used and populated ","Yes","Yes"]
[13,"Are all loops properly terminated?","Yes","Yes"]
[14,"Do all switch statements have a break after each case and terminate with a default case? ","Yes","Yes"]
[15,"Are all macro definitions and arguments surrounded by parentheses? ","Yes","Yes"]
[16,"Has gating been provided for any potential cyclic error/status messages (e.g. log messages, EVRs, FHTs)?","No","No","Could probably throttle cycle slip EVR"]
["Function / Data Usage"]
[{"$f":"A29+1","v":17},"Are declarations grouped into global and static/local?","Yes","Yes"]
[{"$f":"A31+1","v":18},"Have all public header files been analyzed and purged of any private data/definitions/etc.?","Yes","Yes"]
[{"$f":"A32+1","v":19},"Are all variables locally defined unless global or specific visibility is required? ","Yes","Yes"]
[20,"Are global declarations uniquely named?","Yes","Yes"]
[21,"Is the code free of any literals that are not properly documented in the form of macros?","Yes","Yes"]
[22,"Have hardware specific code/data been sufficiently documented?  ","n/a","n/a"]
[23,"Is the precision of floating point numbers sufficient to ensure accuracy?","n/a","n/a"]
[24,"Are all variables initialized before use?","No","No","See autocoder ticket"]
[25,"Have all non-atomic data elements been protected from corruption?","n/a","n/a"]
[26,"Have all function input arguments been validated prior to use?","Yes","Yes"]
[27,"Have unused arguments to functions been documented?","Yes","Yes"]
["Interrupt Service Routines"]
[{"$f":"A41+1","v":28},"Is the code identified as ISR code? (special banner in the function header)","n/a","n/a"]
[{"$f":"A43+1","v":29},"Are all system services called ISR safe? (FSW services, OS services)","n/a","n/a"]
[{"$f":"A44+1","v":30},"Is the ISR free of floating point operations?","n/a","n/a"]
[{"$f":"A45+1","v":31},"Has the ISR been optimized to minimize the length/execution time?","n/a","n/a"]
[{"$f":"A46+1","v":32},"Are any data/structures accessed protected from corruption between ISR and non-ISR code?","n/a","n/a"]
["Critical Regions"]
[{"$f":"A47+1","v":33},"Has the critical region been optimized to minimize the length/execution time?","n/a","n/a","No critical regions"]
[{"$f":"A49+1","v":34},"Has a test been defined/scheduled to verify the critical region's execution time does not exceed the maximum allowable system jitter?","n/a","n/a"]
[{"$f":"A50+1","v":35},"Do all paths guarantee that the critical region is ended?","n/a","n/a"]
[{"$f":"A51+1","v":36},"Could the intent of the critical region being achieved with a less restrictive region (e.g. taskLock)?","n/a","n/a"]
["Maintainability"]
[{"$f":"A52+1","v":37},"Are there sufficient comments to provide understanding of the code?","Yes","Yes"]
[{"$f":"A54+1","v":38},"Is the unit independent of specific hardware wherever possible?","Yes","Yes"]
[{"$f":"A55+1","v":39},"Are any issues/dependencies/constraints related to reuse/reengineering or modifications to the component documented?","No","No"]
[{"$f":"A56+1","v":40},"Does the code accurately reflect the documentation in the design?","Yes","Yes"]
["Other Mission Impacts"]
[41,"Have actions been taken for any issues raised at this meeting that impact other missions in development or already in the operational phase?","No","No"]
["FSW Domain Specific Questions"]
[42,"Have the initialization dependencies, task priorities, message queue depths, etc. been coordinated with the associated project/program integrator(s)?","No","No","Done at instantiation time, not in the component"]
[43,"Have actions been taken for any potential flight rules or modifications to heritage flight rules identified in this review?","No","No"]
[44,"Have actions been taken for any idiosyncrasies identified in this review?  ","No","No"]
["Summary"]
[45,"Have all the action items been captured from this code peer review?",null,"Yes"]
[46,"Is the code ready to progress to the test phase?",null,"Yes"]
["Process Improvement"]
[47,"Were all checklist items (questions and / or comments) clear?","No",null,"Same comments as before"]
[48,"Was the checklist complete?",null,"Yes"]

{"sheet":"Criteria","index":2,"n_rows":5,"n_cols":5,"rows_kept":5}
["Yes/No",null,"Software Development Type Definitions",null,"Software Catetory Definitions"]
["Yes",null,"Reuse - No code changes allowed",null,"1 - Mission/Safety Critical"]
["No",null,"Re-engineer (Functional)   - Code changes planned (<20%)",null,"2 - Mission/Safety Critical Support"]
["n/a",null,"Re-engineer (Data Only)   - Only data changes planned",null,"3 - Non-Mission/Safety Critical Support"]
[null,null,"New - new or >20% modified",null,"4 - Engineering Support"]