User story #241
Updated by André Neto over 9 years ago
Revisit all the source files and make sure that all errors are properly logged.
h1. Source code files modified
h1. Architecture & design review
*Date of the review:* 23/10/2015
*Person who did the review:* André Neto
*Version of architecture & design document:* N/A. As per the original MARTe implementation, this is a support function and does not require formal design in UML.
*Result of review:* N/A
*List of non-conformities:* N/A
h1. Code and documentation review
*Date of the review:* 09/11/2015
*Person who did the review:* Andre' Neto
*Result of review:* PASS
*List of non-conformities:* N/A.
h1. Unit test review
*Date of the review:* 09/11/2015 --/--/2015
*Person who did the review:* Andre' Neto -----
*Result of coverage tests review:* PASS [PASS/FAIL]
*Result of functional tests review:* PASS [PASS/FAIL]
*Result of review:* PASS [PASS/FAIL]
*List of non-conformities:*
As a note, as the number of error reports is increasing, the coverage is struggling to keep up to date. We should try to find a way to tell lcov to accept as covered when an error is reported.
h1. Source code files modified
h1. Architecture & design review
*Date of the review:* 23/10/2015
*Person who did the review:* André Neto
*Version of architecture & design document:* N/A. As per the original MARTe implementation, this is a support function and does not require formal design in UML.
*Result of review:* N/A
*List of non-conformities:* N/A
h1. Code and documentation review
*Date of the review:* 09/11/2015
*Person who did the review:* Andre' Neto
*Result of review:* PASS
*List of non-conformities:* N/A.
h1. Unit test review
*Date of the review:* 09/11/2015 --/--/2015
*Person who did the review:* Andre' Neto -----
*Result of coverage tests review:* PASS [PASS/FAIL]
*Result of functional tests review:* PASS [PASS/FAIL]
*Result of review:* PASS [PASS/FAIL]
*List of non-conformities:*
As a note, as the number of error reports is increasing, the coverage is struggling to keep up to date. We should try to find a way to tell lcov to accept as covered when an error is reported.