search

anuragraghavan / franca

Automatically exported from code.google.com/p/franca
0 stars 0 forks source link

Stackoverflow in case of cyclic interface dependencies #45

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
a cycle in interface inheritance hierarchy results in StackOverFlow

example model:

package p
interface T1 extends T2{}
interface T2 extends T1{} 

relevant trace:
org.franca.core.framework.FrancaHelpers.getAllAttributes(FInterface)
...
org.franca.core.framework.FrancaHelpers.getAllAttributes(FInterface)
...
org.eclipse.xtext.validation.AbstractDeclarativeValidator.internalValidate(EClas
s, EObject, DiagnosticChain, Map<Object, Object>)

Original issue reported on code.google.com by bonge...@gmail.com on 3 Jun 2013 at 11:09

GoogleCodeExporter commented 9 years ago 
The StackOverFlow is now avoided with safe handling in the FrancaHelpers. 
Additionally I have created validators to detect cyclic interface inheritances. 
See commits #5f9de6ddb88a and #f18e6d2ceb7e on addons.

Original comment by szabt...@gmail.com on 26 Jun 2013 at 2:21

GoogleCodeExporter commented 9 years ago 
Fixed in v0.8.9.

Original comment by klaus.birken@gmail.com on 26 Sep 2013 at 2:36