Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Schedule_Window_Record_Term_Array definition #31

Closed
juli1 opened this issue Apr 20, 2015 · 3 comments
Closed

Schedule_Window_Record_Term_Array definition #31

juli1 opened this issue Apr 20, 2015 · 3 comments
Assignees

Comments

@juli1
Copy link
Contributor

juli1 commented Apr 20, 2015

The Get_Module_Schedule_Property returns a Schedule_Window_Record_Term_Array
which has the following definition

type Schedule_Window_Record_Term is record
Partition : Name_Id;
Duration : Time_Type;
Periodic_Processing_Start : Boolean;
end record;

However, the partition part should be a node id with a reference to the partition instead of a name_id. This is a semantic error and it is not possible to solely rely on the name: looking for a subcomponent with this name in the component hierarchy is confusing because several sub-components can have this name.

Thus, it would be better to put a Node_Id and reference the existing referenced component.

@juli1
Copy link
Contributor Author

juli1 commented Apr 30, 2015

ping!

@juli1
Copy link
Contributor Author

juli1 commented May 5, 2015

any update?

yoogx added a commit that referenced this issue May 16, 2015
          name_id to refer to the corresponding partition

          For issue #31
@yoogx
Copy link
Contributor

yoogx commented May 16, 2015

I told you it would take time, as this ticket is dependent on issue #17, which is quite complex. An initial support is now available, and I updated the does and vxworks653 accordingly, ADIRU example passes fine. I close the ticket

@yoogx yoogx closed this as completed May 16, 2015
@yoogx yoogx self-assigned this May 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants