[RkWard-devel] Plugin code running inside local(); please test

[Thomas Friedrichsmeier]

Hi,

On Sunday 04 March 2007 20:40, Thomas Friedrichsmeier wrote:
> a while ago, Prasenjit suggested to place all plugin-generated code inside
> a local() to solve the issue about temporary variables in an elegant way.
> Instead of contemplating the implications for much longer, I have now added
> this to SVN. There were some unexpected complications in rk.set.label(),
> but I hope to have solved those.
>
> Please do *not* modify your plugins to make use of this new functionality,
> yet. Rather, please test whether everything works as expected. If there are
> no further complications, we can switch to this for good, removing all
> cleanup sections, and giving temporary variables nice short names. However,
> until we are reasonably sure, there are no unexpected side-effects, I'd
> like to have an easy way to back out this change, if needed.

I'm pretty conservative about this change, as messing with R environments is 
not my favorite excercise. However, things seem to be looking good, so this 
change is pretty certain to stay. If you feel like it, you can start 
converting your under_development plugins to this new behavior, i.e.:

1) remove the
	function cleanup () {
	...
	}
from the php files.

2) Shorten names like "rk.temp.x" to just "x".

3) In some (few) cases, remove usage of list()s to keep a number of temporary 
variables in a single object.

4) Test, whether the plugin still works correctly after this change.

Later, we'll need to steamroll through all the existing plugins to convert 
those as well, but for the off-chance that further problems turn up, I'd like 
to delay this final step for yet another little while.

Also, if you like - but this is just another attempt of mine to delegate some 
tasks - you could have a look at which sections of the writing plugins 
documentation need to be updated, and perhaps start modifying those 
(doc/en/writing_plugins_introduction.docbook). I won't get around to do this 
for another while.

While I think, it's still too early to plan a release timeline for 0.4.7, I 
also believe we're slowly getting closer to this. So keep this in mind before 
starting "large" new projects, and try to make sure your plugins get proper 
review soon (hint: you might have to nag me about this, I'm starting to loose 
track of what I still need to look at).

Regards
Thomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/rkward-devel/attachments/20070311/f94d7c77/attachment.sig>