Page 1 of 1

issue-#172 adding BOOLEAN parameters to StdInterpreter

Posted: Sat Aug 19, 2017 9:02 am
by Josef Templ
I have created issue https://redmine.blackboxframework.org/issues/172 for adding BOOLEAN params to Dialog.Call.
In addition, this issue fixes a small bug in StdInterpreter by adding a check for correct syntax before calling a procedure.

See the diffs at https://redmine.blackboxframework.org/p ... 982824bd7b.

- Josef

Re: issue-#172 adding BOOLEAN parameters to StdInterpreter

Posted: Sat Aug 19, 2017 9:20 am
by Robert
Isn't the plan to get the known bugs out of 1.7.1, then add new features to 1.7.2?

Re: issue-#172 adding BOOLEAN parameters to StdInterpreter

Posted: Sat Aug 19, 2017 11:17 am
by Josef Templ
Robert wrote:Isn't the plan to get the known bugs out of 1.7.1, then add new features to 1.7.2?
Of course.
The starting point of this issue was a bug fix and the bug is fixed.

Since adding BOOLEAN parameters turned out to be so simple, it is irresistible (at least for me).
It is a trade-off between waiting another year for the next release and
making a little exception in the release procedure.

- Josef

Re: issue-#172 adding BOOLEAN parameters to StdInterpreter

Posted: Mon Aug 21, 2017 8:45 am
by Zinn
There is a typing error in all comments:
- 20170819, center #3172, adding BOOLEAN parameters to StdInterpreter
Helmut

Re: issue-#172 adding BOOLEAN parameters to StdInterpreter

Posted: Tue Aug 22, 2017 8:32 am
by Josef Templ
Zinn wrote:There is a typing error in all comments:
- 20170819, center #3172, adding BOOLEAN parameters to StdInterpreter
Helmut
thanks. Fixed.

see latest diffs at https://redmine.blackboxframework.org/p ... d2c71e8ebf.

see total diffs at https://redmine.blackboxframework.org/p ... 982824bd7b.

- Josef