StackOverflow with TableGrids
Hello,
The following testcase:
- Put 2 TableGrid component on a report
- Set TableGrid1.MasterGrid to TableGrid2
- Set TableGrid2.MasterGrid to TableGrid1
The programm crashes with a stackoverflow while bth grids try to update
each other.
I know that this is something one would "normally" not do an the reason
why the stackoverflow happens is also clear.
But if an enduser uses the report designer it is not completely unlikely
that he produces this combination of property settings and crashes the
programm immediately.
So this combination of TableGrids referencing each other should be
captured by RB and should raise a normal exception or error message when
the user tries to assign this combination.
Ciao,
Ralf
The following testcase:
- Put 2 TableGrid component on a report
- Set TableGrid1.MasterGrid to TableGrid2
- Set TableGrid2.MasterGrid to TableGrid1
The programm crashes with a stackoverflow while bth grids try to update
each other.
I know that this is something one would "normally" not do an the reason
why the stackoverflow happens is also clear.
But if an enduser uses the report designer it is not completely unlikely
that he produces this combination of property settings and crashes the
programm immediately.
So this combination of TableGrids referencing each other should be
captured by RB and should raise a normal exception or error message when
the user tries to assign this combination.
Ciao,
Ralf
This discussion has been closed.
Comments
Thanks for the feedback.
For RB 18.02 I'll add validation to prevent MasterGrid circular references.
For the next major version I'll enhance the object inspector MasterGrid drop
down list to contain only valid candidates. I'll do this for Delphi and RB
object inspectors.
Please update your newsreader to post using your full name.
Best regards,
-
Nard Moseley
Digital Metaphors
www.digital-metaphors.com
Best regards,
Nard Moseley
Digital Metaphors
www.digital-metaphors.com
Thanks for the info. Looking forward to the new version...
Should be fixed by now.
Ciao,
Ralf