issue-#38 revisited: simplify migration to 1.7

Locked

Should we merge topic branch issue-#38 with master?

Yes
5
71%
No
0
No votes
Abstain
2
29%
 
Total votes: 7

Ivan Denisov
Posts: 1700
Joined: Tue Sep 17, 2013 12:21 am
Location: Russia

issue-#38 revisited: simplify migration to 1.7

Post by Ivan Denisov »

This is voting about reverting some VAR parameters to IN for easier migration to 1.7.
http://redmine.blackboxframework.org/issues/38
The discussion
Josef wrote:When porting code from 1.6 to 1.7 I got some compiler messages regarding issue-#38
(passing an IN parameter or read-only variable as a VAR receiver is no longer allowed).
This is not a surprise because this change is the most significant incompatible change of 1.7
and it was of course possible to do the required adaptations.

However, this change may also hit other users and complicate the upgrade to 1.7.
I therefore took a closer look and found out that almost all of the problems
could have been avoided by small refinements in the core system.
Those changes would also avoid changes in some other BB modules
that have been done for #38 and could be undone.

The changes are simply to use IN for the receiver parameter of
methods that don't change the receiver. This approach has
already been applied to module Meta as part of #38.
It would only be needed to also apply it to TextMappers.Formatter
and the various Dialog types such as List or Tree.
There is the demo version:
http://blackboxframework.org/unstable/i ... b1.551.zip

The suggested changes:
http://redmine.blackboxframework.org/pr ... fa1db2c673
Ivan Denisov
Posts: 1700
Joined: Tue Sep 17, 2013 12:21 am
Location: Russia

Re: issue-#38 revisited: simplify migration to 1.7

Post by Ivan Denisov »

Locked