Project

General

Profile

User story #230

Fix namespace dependency propagation on Object.h macros

Added by Ivan Herrero over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
Start date:
28.09.2015
Due date:
% Done:

0%

Estimated time:
Git branch (link):
Git merge to develop (link):
SVN commit (link/?p=rev):

Description

Macros CLASS_REGISTER_DECLARATION and CLASS_REGISTER on Object.h declare/define functions that are dependent on the namespace MARTe. This provokes that users of these macros have to add a "using namespace MARTe;" clause on the client code, which is not compliant with MISRA rule 7-3-4.

Note: The file ObjectTestHelper.h needs to be updated after these macros have been fixed. It will be useful also as a test.

Source code files modified

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

Code and documentation review

N/A

Unit test review

N/A

Associated revisions

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

#230 - Fix namespace dependency propagation on Object.h macros

- Move macros CLASS_REGISTER_DECLARATION and CLASS_REGISTER out of MARTe
namespace declaration.
- Add "MARTe::" prefix on all uses of MARTe symbols inside macros
CLASS_REGISTER_DECLARATION and CLASS_REGISTER.

History

#1 Updated by André Neto over 9 years ago

  • Status changed from New to Code: Impl

#2 Updated by André Neto over 9 years ago

  • Description updated (diff)

#3 Updated by André Neto over 9 years ago

  • Target version set to 0.7

#4 Updated by André Neto over 9 years ago

  • Target version changed from 0.7 to 0.8

#5 Updated by André Neto over 9 years ago

  • Target version changed from 0.8 to 0.9

#6 Updated by André Neto over 9 years ago

  • Status changed from Code: Impl to Code: Rev

#7 Updated by André Neto over 9 years ago

  • Status changed from Code: Rev to Code: Impl

#8 Updated by Ivan Herrero over 9 years ago

  • Status changed from Code: Impl to Unit: Rev

Fixed directly in branch develop (commit d6b1531).

#9 Updated by Ivan Herrero over 9 years ago

  • Status changed from Unit: Rev to Closed

#10 Updated by Ivan Herrero over 9 years ago

  • Description updated (diff)

Also available in: Atom PDF