User story #159

Lint: StringPortable

Added by André Neto almost 10 years ago. Updated over 9 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Target version:

0.2

Start date:

28.06.2015

Due date:

% Done:

Estimated time:

Git branch (link):

Git merge to develop (link):

SVN commit (link/?p=rev):

Description

Verify and implement coding standard using Flexelint.

Source code files modified¶

Source/Core/L0Portability/StringPortable.cpp
Source/Core/L0Portability/StringPortable.h
Test/Core/L0Portability/StringPortableTest.cpp
Test/Core/L0Portability/StringPortableTest.h
Test/GTest/StringPortableGTest.cpp

Architecture & design review¶

Date of the review: 22/07/2015

Person who did the review: Ivan Herrero

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

Code and documentation review¶

Date of the review: 24/07/2015

Person who did the review: Llorenç Capellà

Result of review: PASS (MISRA)

List of non-conformities: N/A

Unit test review¶

Date of the review: 27/07/2015

Person who did the review: Ivan Herrero

Result of coverage tests review: PASS

Result of functional tests review: PASS

Result of review: PASS

List of non-conformities:

Coverage Note: Some errors paths not exercised.

Associated revisions

Revision 433b7819 (diff)
Added by Ivan Herrero over 9 years ago

#159 - Coding standard applied to StringPortable header and source files

Revision 312ad822 (diff)
Added by Giuseppe Ferro over 9 years ago

Implementation and linting of StringPortable done.
#159

Revision 14ccc4bc (diff)
Added by Giuseppe Ferro over 9 years ago

Minor changes in documentation of StringPortable.
#159

Revision 76c16376 (diff)
Added by Giuseppe Ferro over 9 years ago

Concatenate functions overloaded.
Added SetChar function

#159

Revision 75de4636 (diff)
Added by Ivan Herrero over 9 years ago

#159 - Minor formatting corrections (following template rules)

History

#1 Updated by Riccardo Vitelli almost 10 years ago

Target version set to Backlog

#2 Updated by Riccardo Vitelli almost 10 years ago

Target version changed from Backlog to 0.2

#3 Updated by Riccardo Vitelli almost 10 years ago

Status changed from New to Code: Impl

#4 Updated by Riccardo Vitelli over 9 years ago

Status changed from Code: Impl to Code: Rev

#5 Updated by Riccardo Vitelli over 9 years ago

Status changed from Code: Rev to Code: Impl

#6 Updated by Llorenc Capella over 9 years ago

Assignee set to Llorenc Capella

#7 Updated by Llorenc Capella over 9 years ago

Status changed from Code: Impl to Code: Rev

#8 Updated by Llorenc Capella over 9 years ago

Assignee deleted (~~Llorenc Capella~~)

StringPortable imported from develop_svn and linting done.

#9 Updated by Ivan Herrero over 9 years ago

Assignee set to Ivan Herrero

#10 Updated by Ivan Herrero over 9 years ago

Description updated (diff)
Status changed from Code: Rev to Code: Impl
Assignee deleted (~~Ivan Herrero~~)

Code and documentation review made on 433b781.

Note: It needs to be linted again, because the flexelint files were not properly configured (remember to add StringPortable.cpp on marte_flint_files.lnt).