Skip to content
This repository has been archived by the owner on May 18, 2019. It is now read-only.

[NF] Various fixes. #2819

Closed
wants to merge 1 commit into from
Closed

[NF] Various fixes. #2819

wants to merge 1 commit into from

Conversation

perost
Copy link
Member

@perost perost commented Dec 7, 2018

  • Remove the type in Call.CallAttributes, the call itself already
    contains the same type.
  • Improve subscripting of calls such as pre(), where the argument can
    be subscripted instead of the whole call expression.
  • Fix Type.copyDims so that it doesn't create an array with no
    dimensions if the source type is a scalar type.
  • Add simplification 'if ... then x else x' => 'x'.

- Remove the type in Call.CallAttributes, the call itself already
  contains the same type.
- Improve subscripting of calls such as pre(), where the argument can
  be subscripted instead of the whole call expression.
- Fix Type.copyDims so that it doesn't create an array with no
  dimensions if the source type is a scalar type.
- Add simplification 'if ... then x else x' => 'x'.
@OpenModelica-Hudson
Copy link
Member

The tests run correctly according to OpenModelica_TEST_PULL_REQUEST 2018-12-07_16-16-34.

OpenModelica-Hudson pushed a commit that referenced this pull request Dec 7, 2018
- Remove the type in Call.CallAttributes, the call itself already
  contains the same type.
- Improve subscripting of calls such as pre(), where the argument can
  be subscripted instead of the whole call expression.
- Fix Type.copyDims so that it doesn't create an array with no
  dimensions if the source type is a scalar type.
- Add simplification 'if ... then x else x' => 'x'.

Belonging to [master]:
  - #2819
  - OpenModelica/OpenModelica-testsuite#1086
OpenModelica-Hudson pushed a commit to OpenModelica/OpenModelica-testsuite that referenced this pull request Dec 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants