Wrong variables declaration location in functions / procedures / actions #10
Comments
alorence
added a commit
that referenced
this issue
Aug 27, 2013
As explained in issue #10, all CAL actors should declare local variables only in a 'var' block and not anymore in the first statements of the body
alorence
added a commit
that referenced
this issue
Aug 28, 2013
As explained in issue #10, all CAL actors should declare local variables only in a 'var' block and not anymore in the first statements of the body
alorence
added a commit
that referenced
this issue
Aug 28, 2013
As explained in issue #10, all CAL actors should declare local variables only in a 'var' block and not anymore in the first statements of the body
|
Main branches have been updated to fix this issue. I close the ticket, since other branches can easily be fixed by cherry picking one of the three above commits (you can cherry-pick the closest and manually fix other few actors if necessary). |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As mentionned at iv) point on issue orcc/orcc#42, until now Orcc allowed to declare variables in the body of a procedure (or function / action). This is not correct regarding on CAL standard.
A patch to fix this behaviour is ready (see orcc/orcc@6d680800), but before merging it to the master branch, all projects in orc-apps repository should be fixed.
The way to fix is pretty simple but a bit long to perform:
In all actions / functions / procedures, each local variable must be declared in a 'var' block instead of in the body. Each variable declaration must be separated by a comma, instead of a semi-colon.
The text was updated successfully, but these errors were encountered: