Skip to content
This repository has been archived by the owner. It is now read-only.

stack overflow in filtered quantified expression #117

Closed
reteprelief opened this issue Apr 4, 2018 · 4 comments
Closed

stack overflow in filtered quantified expression #117

reteprelief opened this issue Apr 4, 2018 · 4 comments
Assignees

Comments

@reteprelief
Copy link
Contributor

@reteprelief reteprelief commented Apr 4, 2018

filterbug () <=
** "stack overflow" **
	forall  (pc : {pc for (conn:connection)| is_port_connection(conn)}).
	true

The expression should say
forall (pc : {conn for (conn:connection)| is_port_connection(conn)}).

@kfhoech kfhoech self-assigned this Apr 4, 2018
@kfhoech
Copy link
Contributor

@kfhoech kfhoech commented Apr 4, 2018

Thanks for reporting this.

Yes, it appears that the quoted expression is incorrect. However, I cannot locate that construct in the smaccm/smaccm repository.

Can you give indication as to where this is located? Thanks.

@reteprelief
Copy link
Contributor Author

@reteprelief reteprelief commented Apr 4, 2018

The Resolute validator will loop through three methods. This is in the develop branch.

at com.rockwellcollins.atc.resolute.validation.ResoluteJavaValidator.getIdExprType(ResoluteJavaValidator.java:1107)
at com.rockwellcollins.atc.resolute.validation.ResoluteJavaValidator.getExprType(ResoluteJavaValidator.java:999)
at com.rockwellcollins.atc.resolute.validation.ResoluteJavaValidator.getExprType(ResoluteJavaValidator.java:911)
at com.rockwellcollins.atc.resolute.validation.ResoluteJavaValidator.getIdExprType(ResoluteJavaValidator.java:1107)
at com.rockwellcollins.atc.resolute.validation.ResoluteJavaValidator.getExprType(ResoluteJavaValidator.java:999)
at com.rockwellcollins.atc.resolute.validation.ResoluteJavaValidator.getExprType(ResoluteJavaValidator.java:911)
@kfhoech
Copy link
Contributor

@kfhoech kfhoech commented Apr 4, 2018

Oh. This is a validator problem. I thought you were referring to a problem in a library or model.

Duplicated in a91f995.

@kfhoech
Copy link
Contributor

@kfhoech kfhoech commented Apr 6, 2018

Resolved by Pull Request 118.

@kfhoech kfhoech closed this Apr 6, 2018
@kfhoech kfhoech added the v2.3.3 label Jul 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants