Skip to content

Limit arrays#3828

Open
HansOlsson wants to merge 4 commits intomodelica:masterfrom
HansOlsson:LimitArrays
Open

Limit arrays#3828
HansOlsson wants to merge 4 commits intomodelica:masterfrom
HansOlsson:LimitArrays

Conversation

@HansOlsson
Copy link
Collaborator

Closes #3820
I think this covers all of the discussed cases.

As far as I can see it is also consistent with the previous intent, but just missed:

  • Enumeration constructor is part of list that can be called vectorized.
  • Record cast (but not record constructor) explicitly listed that it could be called vectorized
  • Lookup of classes in components restricted it to scalar components; and lookup for classes is described as constructing a component of the class.

@HansOlsson HansOlsson requested a review from henrikt-ma February 6, 2026 14:32
@HansOlsson HansOlsson added this to the 2026-March milestone Mar 4, 2026
@HansOlsson HansOlsson requested a review from qlambert-pro March 5, 2026 15:05
\section{Record Constructor Functions}\label{record-constructor-functions}

Whenever a record is defined, a record constructor function with the same name and in the same scope as the record class is implicitly defined according to the following rules:
Whenever a scalar record is defined, a record constructor function with the same name and in the same scope as the record class is implicitly defined according to the following rules:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just asking: Is there a risk of confusion when speaking of a scalar record? Could it be interpreted as a record just a single member of "size one"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I doubt that, especially as it doesn't work with the rest of the text.

Copy link
Collaborator

@henrikt-ma henrikt-ma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am leaving it for @qlambert-pro to provide a review with approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Modelica array class

2 participants